Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE/#90] Post /posts API 구현 #98

Merged
merged 9 commits into from
Nov 21, 2023
Merged

[BE/#90] Post /posts API 구현 #98

merged 9 commits into from
Nov 21, 2023

Conversation

koomin1227
Copy link
Member

@koomin1227 koomin1227 commented Nov 21, 2023

이슈

체크리스트

  • 요청글과 대여글 분리
  • 대여글에서 사진 없는 경우에도 정상 동작학게 구현
  • S3 handler 활용하여 이미지 파일 업로드 하도록 구현
  • 파일 url DB에 저장

고민한 내용

multipart 형식으로 이미지와 json을 같이 보내면 @Body() 데코레이터로 json 파싱이 되지 않는 문제가 발생하였다.
그래서 커스텀 데코레이터 @MultipartBody() 를 정의 하여 request 객체의 body를 JSON.parse 하여 사용 할 수 있도록 수정하였다.

스크린샷

@koomin1227 koomin1227 added BE BE 관련 post-create 게시글 등록 labels Nov 21, 2023
@koomin1227 koomin1227 self-assigned this Nov 21, 2023

return data ? JSON.parse(body?.[data]) : body;
},
);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

커스텀 데코레이터를 사용해서 request.body 를 파싱한 과정이 좋네요~

@koomin1227 koomin1227 merged commit bf6f3c7 into BE Nov 21, 2023
1 check passed
@koomin1227 koomin1227 linked an issue Nov 21, 2023 that may be closed by this pull request
@koomin1227 koomin1227 deleted the BE-PostPostsAPI-#90 branch November 21, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE BE 관련 post-create 게시글 등록
Projects
None yet
Development

Successfully merging this pull request may close these issues.

POST /posts API 구현
2 participants