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

Feat/apply #33

Merged
merged 15 commits into from
Feb 6, 2024
Merged

Feat/apply #33

merged 15 commits into from
Feb 6, 2024

Conversation

sycuuui
Copy link
Contributor

@sycuuui sycuuui commented Feb 2, 2024

#️⃣ 연관된 이슈

ex) #26 #30 #32

📝 작업 내용

category - 지원서,지원 요청시 category정보 코드 구현
Apply entity recruit_role 추가
지원 시 에러 코드 구현
공고 모집 취소,연장,마감 요청 api 기능 구현 (취소,마감 일시적으로 주석처리)
모집 완료 시 모집 파트 인원 감소 코드 구현

스크린샷 (선택)

스크린샷 2024-02-02 오후 5 20 22 스크린샷 2024-02-02 오후 5 31 07

💬 리뷰 요구사항(선택)

@sycuuui sycuuui added feat 기능 개발 시 fix 기존 작성된 코드 수정 시 docs 문서 작업 시 labels Feb 2, 2024
@sycuuui sycuuui self-assigned this Feb 2, 2024
@sycuuui sycuuui removed the docs 문서 작업 시 label Feb 2, 2024
Copy link
Contributor

@dl-00-e8 dl-00-e8 left a comment

Choose a reason for hiding this comment

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

PR 내용을 보면 지원할 때, 지원 수가 줄어드는 방식으로 표현하신 걸로 이해했습니다!
생각해보았을 때, 팀장이 선발 완료했을 때 해당 수가 줄어들어야 하는데 제가 코드 레벨에서 이 방식으로 적용된 것이 맞는지, 맞다면 코드 위치좀 알려주실 수 있나요?

@dl-00-e8
Copy link
Contributor

dl-00-e8 commented Feb 4, 2024

PR 내용을 보면 지원할 때, 지원 수가 줄어드는 방식으로 표현하신 걸로 이해했습니다! 생각해보았을 때, 팀장이 선발 완료했을 때 해당 수가 줄어들어야 하는데 제가 코드 레벨에서 이 방식으로 적용된 것이 맞는지, 맞다면 코드 위치좀 알려주실 수 있나요?

@sycuuui

@sycuuui
Copy link
Contributor Author

sycuuui commented Feb 5, 2024

제가 잘못 생각했에요 ㅠㅠ 팀장이 선발 완료했을 때 해당 수가 줄어들어야 하는 코드로 수정 후 다시 올리겠습니다!

Copy link
Member

@yumzen yumzen left a comment

Choose a reason for hiding this comment

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

수고하셨습니당

Copy link
Contributor

@dl-00-e8 dl-00-e8 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@dl-00-e8 dl-00-e8 merged commit 6dfaae6 into main Feb 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 개발 시 fix 기존 작성된 코드 수정 시
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants