-
Notifications
You must be signed in to change notification settings - Fork 3
[feat] 라우트 히스토리 제거를 위해 replace옵션추가 #1053
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
[feat] 라우트 히스토리 제거를 위해 replace옵션추가 #1053
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| 코호트 / 파일 | 변경 요약 |
|---|---|
탭 네비게이션 히스토리 처리 수정 frontend/src/pages/ClubDetailPage/ClubDetailPage.tsx |
INTRO 및 PHOTOS 탭 핸들러의 setSearchParams 호출에 { replace: true } 옵션 추가로 탭 선택 시 히스토리 스택을 대체하도록 변경 |
코드 검토 예상 소요 시간
🎯 2 (Simple) | ⏱️ ~10분
관련 이슈
- [feature] MOA-525 소개내용 활동사진 뒤로가기 문제를 해결한다 #1052: 본 PR에서 소개내용 및 활동사진 탭의 뒤로가기 버튼 동작을 수정하는 히스토리 관리 변경사항으로 동일한 기능 동작 개선을 다룸
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | PR 제목이 변경사항의 핵심을 명확하게 반영하고 있습니다. 라우트 히스토리 제거를 위해 replace 옵션을 추가한다는 실제 코드 변경사항과 일치합니다. |
| Linked Issues check | ✅ Passed | 코드 변경사항이 연결된 이슈 MOA-525의 목적을 충족하고 있습니다. 소개/활동사진 탭 클릭 시 { replace: true } 옵션을 적용하여 라우트 히스토리 누적 문제를 해결했습니다. |
| Out of Scope Changes check | ✅ Passed | 모든 변경사항이 라우트 히스토리 제거라는 정해진 범위 내에 있습니다. ClubDetailPage.tsx의 두 클릭 핸들러에만 제한된 변경으로 범위를 벗어난 수정이 없습니다. |
| 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
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
This reverts commit 42a1426.
suhyun113
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
사용하면서 불편함을 느꼈었는데, 탭 클릭 시 히스토리가 쌓여서 생긴 문제였군요..
같은 페이지 내 탭 전환은 reaplce: true로 상태만 변경해주는거 깔끔하네여~
#️⃣연관된 이슈
📝작업 내용
상세페이지 소개내용 | 활동사진 탭 클릭 시 라우트가 쌓여서 메인으로 돌아가려면 여러번 뒤로가기를 해야 헀습니다.
{ replace: true }를 추가해서 라우트 히스토리가 쌓이지 않도록 했습니다.중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
버그 수정
✏️ Tip: You can customize this high-level summary in your review settings.