Skip to content

Comments

fix: applications 중복 활성 지원서 조회 시 500 오류 방지#682

Merged
lsy1307 merged 2 commits intomasterfrom
hotfix/260220
Feb 20, 2026
Merged

fix: applications 중복 활성 지원서 조회 시 500 오류 방지#682
lsy1307 merged 2 commits intomasterfrom
hotfix/260220

Conversation

@lsy1307
Copy link
Contributor

@lsy1307 lsy1307 commented Feb 20, 2026

관련 이슈

작업 내용

  • ApplicationRepository의 사용자/학기 활성 지원서 조회를 단건 가정 조회에서 최신 1건 조회로 변경했습니다.
  • 리팩터링에 맞춰 테스트 코드도 동기화했습니다.

특이 사항

  • 근본 원인(동시성/데이터 정합성)은 별도로 DB 제약(활성 지원서 유니크 보장) 및 데이터 정리 작업이 필요합니다.

리뷰 요구사항 (선택)

- 활성 지원서 조회를 최신 1건 기준 메서드로 변경

- 지원서 제출 로직과 테스트 코드를 변경된 조회 메서드명에 맞춰 동기화
@lsy1307 lsy1307 self-assigned this Feb 20, 2026
@lsy1307 lsy1307 added the 버그 Something isn't working label Feb 20, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 20, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/260220

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@lsy1307 lsy1307 changed the title hotfix: applications 중복 활성 지원서 조회 시 500 오류 방지 fix: applications 중복 활성 지원서 조회 시 500 오류 방지 Feb 20, 2026
Copy link
Contributor

@Gyuhyeok99 Gyuhyeok99 left a comment

Choose a reason for hiding this comment

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

최신순 한 개니까 문제없을 거 같네요! 감사합니다~
이건 이번 모의지원 끝나고 천천히 고치죠~ 괜히 장애나면 안되니까요!

@Gyuhyeok99
Copy link
Contributor

혹시 여유가 되신다면 todo 하나만 추가해서 머지해주셔도 좋을 거 같습니다..!

- 최신 1건 선택 로직이 임시 회피임을 주석으로 명확화

- DB 유일성 제약 도입 후 우회 조회 제거 필요사항을 기록
@lsy1307 lsy1307 merged commit a098ff3 into master Feb 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants