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

RAC-277 refactor : 수정 및 편의 사항 추가 #92

Merged
merged 16 commits into from
Jan 14, 2024
Merged

Conversation

ywj9811
Copy link
Member

@ywj9811 ywj9811 commented Jan 10, 2024

🦝 PR 요약

수정 및 편의 사항 추가

✨ PR 상세 내용

  • JWT관련 예외 응답 코드 변경
  • 결제가 실패하는 경우 멘토링 삭제할 수 있도록 API 추가
  • 매월 Salary 자동 생성시 Account 정보 추가
  • �Salary 자동 생성시 조회 쿼리 발생 감소 - 성능개선
  • 매월 정산일 Slack 메시지 전송 -> 발생 쿼리 감소
  • 로그 서버 연결 실패시(로그 서버 이슈 등등) 슬랙에 알림 전송 후 정산 진행하도록 수정

🚨 주의 사항

application.yml에 변경사항이 있습니다!
환불 관련된 로직이 추가되어 있는데 이 부분은 사용하지 않는 부분입니다 (자꾸 빌드가 실패해서 그냥 다 커밋해버렸어요..)

✅ 체크 리스트

  • 리뷰어 설정했나요?
  • Label 설정했나요?
  • 제목 양식 맞췄나요? (ex. RAC-1 feat: 기능 추가)
  • 변경 사항에 대한 테스트 진행했나요?

@ywj9811 ywj9811 added ✨ Feature 새로운 기능 추가 🛠️ Refactor 리팩토링 labels Jan 10, 2024
@ywj9811 ywj9811 requested a review from ay-eonii January 10, 2024 17:51
@ywj9811 ywj9811 self-assigned this Jan 10, 2024
@ywj9811
Copy link
Member Author

ywj9811 commented Jan 10, 2024

로컬은 빌드가 잘 되는데... 모든 테스트가 다 되는데 왜...

@ywj9811 ywj9811 added the 🐞 BugFix 오류 수정 label Jan 13, 2024
Copy link
Collaborator

@ay-eonii ay-eonii left a comment

Choose a reason for hiding this comment

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

코멘트 한 번만 확인해주세용

슬랙에 오류 보내도록 해주셔서 감사합니다 🙇‍♀️
결제가 실패하는 경우 멘토링 삭제할 수 있도록 대비하는 거 짱입니다 👍👍

@ywj9811
Copy link
Member Author

ywj9811 commented Jan 14, 2024

해당 부분들 다 고쳐서 다시 올렸어요!

@ay-eonii ay-eonii self-requested a review January 14, 2024 13:45
Copy link
Collaborator

@ay-eonii ay-eonii left a comment

Choose a reason for hiding this comment

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

확인했습니다! 좋아용~ 굿

@ywj9811 ywj9811 merged commit be006b2 into develop Jan 14, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix 오류 수정 ✨ Feature 새로운 기능 추가 🛠️ Refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants