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

#250 [refactor] 엔드포인트 변경, 디자이너 제안 상세보기 response 도메인별로 분리 #251

Merged
merged 15 commits into from
Feb 24, 2024

Conversation

hellozo0
Copy link
Member

관련 이슈번호

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

  • 2h / 40m

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

  • 엔드포인트변경
  • 디자이너 제안 상세보기 도메인 별로 객체 묶어 response 재조합

어떻게 해결했나요?

*먼저 엔드포인트 변경
스크린샷 2024-02-22 오후 4 10 06
스크린샷 2024-02-22 오후 4 10 23


* 제안서 상세보기 내용 재조합 스크린샷 2024-02-22 오후 4 05 16 스크린샷 2024-02-22 오후 4 05 30

@hellozo0 hellozo0 self-assigned this Feb 22, 2024
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.

작업하시느라 고생하셨습니다!
코맨트 확인해주세요!
또한, Controller도 Retrieve 와 Register 로 분리하면 좋을 것 같습니다!

@Operation(summary = "[JWT] 디자이너 메인 뷰 조회", description = "디자이너 메인 뷰 조회 API입니다.")
@ApiResponses({
@ApiResponse(responseCode = "200", description = "디자이너 메인뷰 조회 성공", content = @Content(schema = @Schema(implementation = DesignerMainResponse.class))),
@ApiResponse(responseCode = "401", description = "인증 오류 입니다.", content = @Content(schema = @Schema(implementation = ErrorResponse.class))),
@ApiResponse(responseCode = "500", description = "서버 내부 오류 입니다.", content = @Content(schema = @Schema(implementation = ErrorResponse.class))),
})
@GetMapping(value = "/designer")
@GetMapping()
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
@GetMapping()
@GetMapping

p3 소괄호를 넣지 않아도 /application 으로 엔드포인트가 적용 됩니다!

Comment on lines 58 to 59
@PostMapping()
@SecurityRequirement(name = "JWT Auth")
Copy link
Member

Choose a reason for hiding this comment

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

p3
소괄호를 넣지 않아도 /model 로 엔드포인트가 적용 됩니다!

@Operation(summary = "[JWT] 모델 메인 뷰 조회", description = "모델 메인 뷰 조회 API입니다.")
@ApiResponses({
@ApiResponse(responseCode = "200", description = "모델 메인뷰 조회 성공", content = @Content(schema = @Schema(implementation = ModelMainOfferResponse.class))),
@ApiResponse(responseCode = "401", description = "인증 오류 입니다.", content = @Content(schema = @Schema(implementation = ErrorResponse.class))),
@ApiResponse(responseCode = "500", description = "서버 내부 오류 입니다.", content = @Content(schema = @Schema(implementation = ErrorResponse.class))),
})
@GetMapping("/model")
@GetMapping()
Copy link
Member

Choose a reason for hiding this comment

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

p3
소괄호를 넣지 않아도 /offer 로 엔드포인트가 적용 됩니다!

@hellozo0
Copy link
Member Author

네에엡! 완료 했습니다~!

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 c4f0265 into develop Feb 24, 2024
1 check passed
@hellozo0 hellozo0 deleted the refactor/#250 branch February 24, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants