Skip to content

Conversation

@enunsnv
Copy link
Contributor

@enunsnv enunsnv commented Feb 1, 2026

관련 이슈

작업 내용

로그인 O 로그인 X

지원자 현황 확인 진입 시, 수정 전에는 에러 토스트가 2가지가 나오던 것을 수정 후 로그인 확인에 실패한 경우에 대해서 올바를 에러 토스트 1가지만 나오도록 수정하였습니다. 따라서 로그인 후 동일 페이지 진입 시에는 로그인 관련 에러 토스트가 더 이상 나오지 않는 것으로 확인했습니다!

특이 사항

로그인에는 성공했지만 점수 공유 현황의 목록이 나오지 않는 경우를 테스트 해보기 위해 개발자 도구에서 applications api의 요청 URL을 임의로 차단하는 방식을 이용했습니다. 테스트 해 본 결과, 에러 토스트 메세지로 'Network Error'를 띄우고 /university/application/apply 로 이동하여 사용자에게 지원서를 입력하도록 작동하고 있는 것을 확인했습니다! 해당 과정에서 추가로 수정할 부분이 있다면 알려주세요!

@vercel
Copy link

vercel bot commented Feb 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connection-web Ready Ready Preview, Comment Feb 1, 2026 11:49am

@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Walkthrough

  1. 오류 알림 제거
    • ScorePageContent.tsx 파일에서 데이터 로딩 중 발생하는 오류 시 표시되던 toast.error 알림이 제거되었습니다. 오류 처리 흐름이 사용자에게 보이는 알림에서 무음 리다이렉트로 변경되었으며, 라우터 네비게이션은 /university/application/apply로 동일하게 유지됩니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • wibaek
  • khwww
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 핵심 변경사항을 명확하게 설명합니다: 중복 토스트 메시지 제거.
Description check ✅ Passed PR 설명이 관련 이슈, 작업 내용, 테스트 결과를 포함하며 템플릿 구조를 따릅니다.
Linked Issues check ✅ Passed 코드 변경사항이 #369의 요구사항을 충족합니다: 중복 토스트 제거 및 올바른 에러 처리 유지.
Out of Scope Changes check ✅ Passed 변경사항이 모두 #369 해결과 직접 관련되어 있으며 범위를 벗어나지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/toast-deduplication

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.

Copy link
Contributor

@manNomi manNomi left a comment

Choose a reason for hiding this comment

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

아 저기에 증복 토스트가 있었군요!!
은정님 진짜 최고 !!

@enunsnv enunsnv merged commit f478b0e into main Feb 1, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

지원자 현황 페이지 진입시 애러 토스트 두개

3 participants