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

[개선] Reject 및 Approve 디비에 저장 및 반려된 지원자는 다시 원서를 제출 할 수 있어야함 #90

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

jyj1289
Copy link
Member

@jyj1289 jyj1289 commented Jun 11, 2024

  • 반려된 원서를 재제출 할 수 있게 하기 위해서 Reject를 디비에 저장할 수 있게 트랜잭션을 달았어요

🎫 관련 이슈

close #88


📄 개요

Reject와 Approve도 디비에 저장되도록 변경했습니다.


🔨 작업 내용

  • Reject와 Approve에 트랜잭션 어노테이션을 달아서 디비에 반영되게 했습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • [ x 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

- 반려된 원서를 재제출 할 수 있게 하기 위해서 Reject를 디비에 저장할 수 있게 트랜잭션을 달았어요
Copy link

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@jyj1289 jyj1289 merged commit e906600 into develop Jun 11, 2024
3 checks passed
@jyj1289 jyj1289 deleted the pref/#88 branch June 11, 2024 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant