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

refactor: 잡포스트 리팩토링 #77

Closed
wants to merge 10 commits into from
Closed

refactor: 잡포스트 리팩토링 #77

wants to merge 10 commits into from

Conversation

YooJHyun
Copy link
Collaborator

@YooJHyun YooJHyun commented Oct 4, 2024

closed #이슈번호

#64

#️⃣연관된 이슈

☑️ PR 유형

  • 새로운 기능 추가
  • 버그 수정
  • 문서 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 주석 추가 및 수정
  • 파일 혹은 폴더 삭제
  • 파일 혹은 폴더명 수정
  • CSS 등 사용자 UI 디자인 변경

📝 작업 내용

  • Apply, JobPost advice 적용
  • JobPost 페이징 적용
  • JobPost 지원시 'Id님 지원 완료' 메시지 노출
  • 회원이 본인의 지원서만 볼 수 있도록 변경

1회 사이클 후 구현 예정 기능

  • 게시글 작성자의 다른 글 조회하는 기능
  • 게시글의 모든 지원자를 게시자가 확인하는 기능
  • 게시자가 지원자의 상태를 변경하는 기능 ex) pass, hold, denied 등등

✅ 피드백 반영사항

💬 리뷰 요구사항

@YooJHyun YooJHyun added the 🔧 refactor 코드 리팩토링 관련 label Oct 4, 2024
@YooJHyun YooJHyun linked an issue Oct 4, 2024 that may be closed by this pull request
JobPost Swagger 적용 변경 및 JobPost 추가 구현 #64
Open
2 tasks
ghtndl
ghtndl approved these changes Oct 4, 2024
View reviewed changes
Copy link
Collaborator

@ghtndl ghtndl left a comment

Choose a reason for hiding this comment

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

확인했습니다 고생하셨습니다

juwon-code
juwon-code requested changes Oct 4, 2024
View reviewed changes
Copy link
Collaborator

@juwon-code juwon-code left a comment

Choose a reason for hiding this comment

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

작성하느라 수고하셨습니다! 코드 약간만 수정 부탁드릴께요^^

src/main/java/com/prgrms2/java/bitta/apply/controller/advice/ApplyControllerAdvice.java
Copy link
Collaborator

Choose a reason for hiding this comment

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

MethodArgumentNotValidException, ConstraintViolationException은 GlobalControllerAdvice에서 처리하고 있으므로 제외해야 할 것 같습니다.

src/main/java/com/prgrms2/java/bitta/apply/dto/ApplyDTO.java
Copy link
Collaborator

Choose a reason for hiding this comment

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

jobPostId와 memberId의 @NotNull 메시지가 동일한 이유가 궁금합니다.
또한 @Min()으로 설정해주셨는데, 관련 에러 메시지도 명시해주셨으면 합니다.

src/main/java/com/prgrms2/java/bitta/apply/service/ApplyService.java Show resolved Hide resolved
src/main/java/com/prgrms2/java/bitta/jobpost/controller/JobPostController.java
Copy link
Collaborator

Choose a reason for hiding this comment

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

데이터 검증을 사용하는 어노테이션에는 @Valid와 @Validated가 있습니다.
약간 수정이 필요한 부분이 있는 것 같습니다. 아래 링크를 참고하여 수정해주셨으면 합니다.
필요하시다면 예시로 FeedController를 참고해주세요.
https://medium.com/sjk5766/valid-vs-validated-%EC%A0%95%EB%A6%AC-5665043cd64b

src/main/java/com/prgrms2/java/bitta/jobpost/controller/advice/JobPostControllerAdvice.java
Copy link
Collaborator

Choose a reason for hiding this comment

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

마찬가지로 MethodArgumentNotValidException, ConstraintViolationException은 GlobalControllerAdvice에서 처리하고 있으므로 제외해야 할 것 같습니다. 만약 여러 ControllerAdvice가 같은 Exception을 처리하게 될 경우, 잘 모르겠지만 Ambiguous 관련 오류가 발생할 것 같습니다.

src/main/java/com/prgrms2/java/bitta/jobpost/dto/JobPostDTO.java
Copy link
Collaborator

Choose a reason for hiding this comment

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

startDate와 endDate의 @FutureOrPresent 날짜 범위가 잘못 설정되어 있는 것 같습니다.
startDate와 endDate는 커스텀 가능하게 하는 게 낫지 않을까요?
후에 해당 게시글을 참고할 때 DTO에서 오류가 발생할 수 있으니까요.

src/main/resources/application.properties
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 파일 크기 제한을 삭제하신 이유가 있으실까요?

@deveunhwa deveunhwa changed the title Dev/refactor/jobpost/#64 refactor: 잡포스트 리팩토링 Oct 4, 2024
@YooJHyun YooJHyun closed this Oct 4, 2024
@YooJHyun YooJHyun reopened this Oct 4, 2024
@YooJHyun YooJHyun closed this Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juwon-code juwon-code juwon-code requested changes

@ghtndl ghtndl ghtndl approved these changes

@jungyoeal jungyoeal Awaiting requested review from jungyoeal

@Preta3418 Preta3418 Awaiting requested review from Preta3418

@deveunhwa deveunhwa Awaiting requested review from deveunhwa

Assignees
No one assigned
Labels
🔧 refactor 코드 리팩토링 관련
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

JobPost Swagger 적용 변경 및 JobPost 추가 구현
3 participants
@YooJHyun @ghtndl @juwon-code