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

구인공고 리스트 API 모집여부 출력 및 모집 중 필터링 추가 #298

Merged
merged 12 commits into from
Nov 18, 2023

Conversation

baekhangyeol
Copy link
Collaborator

@baekhangyeol baekhangyeol commented Nov 12, 2023

🛠️ 변경사항

Recruitment 도메인에 isRecruiting(Boolean) 필드 추가
isRecruitment의 메서드를 정의하여 endDate의 값에 따라 동적으로 변경하도록 구현
구인 게시글 리스트 조회 API에 isRecruiting으로 필터링되도록 수정
구인 게시글 상세 조회 API와 구인 게시글 리스트 조회 API에 모집중 여부 출력되도록 구현

imageimage
image
image

☝️ 유의사항

👀 참고자료

❗체크리스트

  • 하나의 메소드는 최소의 기능만 하도록 설정했나요?
  • 수정 가능하도록 유연하게 작성했나요?
  • 필요 없는 import문이나 setter 등을 삭제했나요?
  • 기존의 코드에 영향이 없는 것을 확인하였나요?

Copy link

github-actions bot commented Nov 12, 2023

Unit Test Results

85 tests  ±0   85 ✔️ ±0   3s ⏱️ ±0s
17 suites ±0     0 💤 ±0 
17 files   ±0     0 ±0 

Results for commit e707f7b. ± Comparison against base commit 9a8b1c3.

♻️ This comment has been updated with latest results.

@baekhangyeol baekhangyeol changed the title 구인공고 리스트 API에 모집여부 출력 및 모집 중 필터링 추가 구인공고 리스트 API 모집여부 출력 및 모집 중 필터링 추가 Nov 13, 2023
Copy link
Collaborator

@youKeon youKeon left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!! 리뷰 확인 부탁드립니다 ;)

@baekhangyeol baekhangyeol merged commit acd257d into feature Nov 18, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 구인공고 리스트 API에 모집여부 출력 및 모집 중 필터링 추가
2 participants