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.

Sign up for GitHub

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

Jump to bottom

#155 [fix] Content Type, Json 으로 나눠서 받을 수 있도록 API 수정 #157

Merged
merged 5 commits into from
Jan 16, 2024
Merged

#155 [fix] Content Type, Json 으로 나눠서 받을 수 있도록 API 수정 #157

merged 5 commits into from
Jan 16, 2024

Conversation

hellozo0
Copy link
Member

@hellozo0 hellozo0 commented Jan 16, 2024 •
edited
Loading

관련 이슈번호

해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)

  • 30m / 30m

해결하려는 문제가 무엇인가요?

  • 하나의 ContentType -> @RequestPart로 쪼개서 Multipart 2개와 json으로 받도록 수정

어떻게 해결했나요?

  • Request의 String -> ENUM으로 변경
  • Parameter 에 MultipartFILE 꺼내기

@hellozo0 hellozo0 added the fix label Jan 16, 2024
@hellozo0 hellozo0 requested review from KWY0218 and pkl0912 January 16, 2024 17:23
@hellozo0 hellozo0 self-assigned this Jan 16, 2024
pkl0912
pkl0912 approved these changes Jan 16, 2024
View reviewed changes
Copy link
Contributor

@pkl0912 pkl0912 left a comment

Choose a reason for hiding this comment

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

화이팅 !!!! 현주는 할 수 잇서

KWY0218
KWY0218 approved these changes Jan 16, 2024
View reviewed changes
Copy link
Member

@KWY0218 KWY0218 left a comment

Choose a reason for hiding this comment

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

좋습니다!!!

@hellozo0 hellozo0 merged commit 4cc10f3 into develop Jan 16, 2024
1 check passed
@hellozo0 hellozo0 deleted the fix/#155 branch January 16, 2024 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pkl0912 pkl0912 pkl0912 approved these changes

@KWY0218 KWY0218 KWY0218 approved these changes

Assignees

@hellozo0 hellozo0

Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[fix] 지원서 작성하기 Multipart RequestBody로직 변경
3 participants
@hellozo0 @KWY0218 @pkl0912