Repository navigation
[Feat/#114] 홈 화면 버튼 라우터 연결 - #117
Open
sangrae2325 wants to merge 22 commits into
Open
sangrae2325 wants to merge 22 commits into
sangrae2325 wants to merge 22 commits into
Conversation
# Conflicts: # docs/conventions/wds-component-usage.md
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 |
sangrae2325
marked this pull request as ready for review
October 11, 2026 08:06
github-actions
Bot
requested review from
jjunh33,
leegain1,
tnals0924 and
xeoxxn
October 11, 2026 08:06
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
🎯 해결하려는 문제가 무엇인가요?
홈 화면(#93)은 화면만 만들고 다른 화면으로 넘어가는 버튼·카드를 대부분 연결하지 않았다.
❓ 왜 해결해야 하나요?
홈에서 공지·신청·대여·내 내역으로 들어갈 수 없어서, 홈이 요약만 보여 주고 끝난다.
실제 동작 영상
다른 화면들 모두 머지/반영후에 추가하겠습니다
⭐ 어떻게 해결했나요?
1. 빌릴게 대여/반납 탭을 경로로 (
31716a2)/bililge, 반납/bililge/returns. 행사 목록([Feat/#115] 행사 신청내역 화면 구현 (목록·빈 상태·상세·신청취소) #116)과 같은 방식으로 같은 화면이 주소로 탭을 고르고, 탭 전환은 replace/bililge/returns에서도 "빌릴게"가 선택된다2. 목데이터 정리 (
9e1be5f)noticeId·eventId·applicationId)와 신청 카드 종류(행사/사물함) 추가3. 홈 이동 연결 (
86b90fc)/notices/:noticeId신청하기→ 행사 신청신청하기둘 다 → 사물함 신청/lockers/applyN건 더보기/ 빈 상태대여하러 가기/bililge/returns//bililgeN건 더보기/ 빈 상태행사 둘러보기/events/applications/:applicationId//events/applications//events/my/fee·/my/feedbacks·/my/locker신청내역 보기도 없는/my/applications대신/my/locker로viewTransition), 하단 탭 화면으로는 바로 전환4. 홈에서 반납 신청 (
8e8a19a)반납 신청을 누르면 홈에서 바로 확인 모달 → 완료 토스트useBililgeReturnRequest훅으로 꺼내 두 화면이 같이 쓴다1133:50014)대로수정→닫기로 바로잡았다5. 문서 (
f51baa8) —wds-component-usage.md홈 절에 연결 표와 규칙 추가🧩 이 PR의 한계 & 트레이드오프
⛓️ 기존 기능에 미치는 영향
수정→닫기(동작은 그대로 모달만 닫음)신청내역 보기가 준비 중 화면 대신 사물함 배정 상태로 간다🔀 Edge Case & 실패 시나리오
📋 검토한 대안과 선택 이유
💬 리뷰 포인트
[c]빌릴게·행사 탭 경로(/bililge/returns,/events/applications) 이름[c]반납 모달닫기문구 변경[a]목데이터 간 문구 차이는 API 연동 때 해소