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

[Feature] 이동봉사 중개 이동봉사 신청 확정, 반려 API 구현 #69

Merged
merged 4 commits into from
Nov 15, 2023

Conversation

kyeong-hyeok
Copy link
Member

💡 연관된 이슈

closte #68

📝 작업 내용

  • 이동봉사 중개 이동봉사 신청 확정 API 구현
  • 이동봉사 중개 이동봉사 신청 확정 Controller 테스트 코드 추가
  • 이동봉사 중개 이동봉사 신청 반려 API 구현
  • 이동봉사 중개 이동봉사 신청 반려 Controller 테스트 코드 추가

💬 리뷰 요구 사항

이동봉사 신청 확정 API를 처음에는 POST로 했다가! 공고와 신청의 상태 변화가 있지만, 새로운 리소스는 생성하지 않아 PATCH로 바꿨습니다!

@kyeong-hyeok kyeong-hyeok added ✨ Feature 기능 개발 Priority: Medium 우선순위 중간 🐯 Koeyhk 담당자 labels Nov 15, 2023
@kyeong-hyeok kyeong-hyeok linked an issue Nov 15, 2023 that may be closed by this pull request
4 tasks
@kyeong-hyeok kyeong-hyeok merged commit 405a9e1 into develop Nov 15, 2023
1 check passed
@kyeong-hyeok kyeong-hyeok deleted the feat/68-application-confirm-cancel-api branch November 15, 2023 06:04
@hojeong2747
Copy link
Member

오오! 항상 PATCH를 쓰려고 할 때는 명확한 기준이 뭘까 고민을 했었는데 '새로운 리소스 생성'을 앞으로 생각해 봐야겠어요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🐯 Koeyhk 담당자 Priority: Medium 우선순위 중간
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 이동봉사 중개 이동봉사 신청 확정, 반려 API 구현
2 participants