Repository navigation
[Feat/#111] 내 내역 확인 화면 (열린피드백 작성내역·학생회비 납부내역·사물함 배정 상태) - #113
Open
sangrae2325 wants to merge 8 commits into
Open
sangrae2325 wants to merge 8 commits into
sangrae2325 wants to merge 8 commits into
Conversation
|
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 9, 2026 16:08
github-actions
Bot
requested review from
jjunh33,
leegain1,
tnals0924 and
xeoxxn
October 9, 2026 16:08
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.
#️⃣연관된 이슈
🎯 해결하려는 문제가 무엇인가요?
홈의 내 정보 목록(학생회비·열린피드백·사물함)에서 들어갈 화면이 없었다. Figma 홈 섹션(
3147:145181)의 내 내역 화면 3종을 목데이터로 구현한다./my/feedbacks3147:147748, 빈 상태3147:147818/my/feedbacks/:feedbackId3147:147756, 답변대기3147:147779/my/fee3147:147860, 확인필요3147:147879, 납부완료3147:147841/my/locker3147:148357, 빈 상태3147:148331/my/locker/location❓ 왜 해결해야 하나요?
홈에서 자기 납부·피드백·사물함 상태를 요약만 보고 자세한 내역을 확인할 곳이 없다. 홈 → 각 화면 라우터 연결 이슈를 하기 전에 들어갈 화면이 먼저 있어야 한다.
실제뷰
⭐ 어떻게 해결했나요?
1. 데이터는 main 규칙대로 (
entities/<도메인>+mockupApi+use()+ 화면별 스켈레톤)entities/feedbacks:fetchMyFeedbacks·fetchMyFeedback(id). 기존features/feedbacks/constants/feedbacks.ts의 피드백 타입·목데이터도 여기로 옮겨 한 곳에 둔다. 열린피드백 목록·상세·검색은 import 경로만 바뀌었다entities/fee:fetchFeePayments— 학기별 여러 건, 최신순entities/lockers:fetchMyLockerAssignment— 신청 안 했으면null*_MOCK_STATE값을 바꾸면 빈 상태·납부 상태별 화면을 볼 수 있다2. 열린피드백 작성내역 (
features/feedbacks)열린피드백 작성하기→/feedbacks/new해당 회차 답변 모아보기→/feedbacks/:id, 답변대기면 빈 상태 +열린피드백 보러 가기→/feedbacks**굵게**처리(renderAnswerText)를FeedbacksAnswerText로 꺼내 모아보기 상세와 같이 쓴다3. 학생회비 납부내역 (
features/fee, 신규)문의하기(학생회 카카오톡 채널)4. 사물함 배정 상태 + 위치 보기 (
features/lockers)사물함 위치 보기)LockersMapView로 꺼내 두 화면이 같이 쓴다5. 사물함 컴포넌트 이름 정리 (refactor 커밋)
Lockers뒤에Locker가 또 붙어 겹치던 이름을 정리했다. 동작 변화 없음LockersLockerCell·Grid·Legend·MinimapLockersCell·LockersGrid·LockersLegend·LockersMinimapLockersLockerSelectScreenLockersCellSelectScreen6. 문서 —
wds-component-usage.md에 내 내역 화면 WDS 매핑 절 추가🧩 이 PR의 한계 & 트레이드오프
문의하기는 아직 동작하지 않는다. 학생회 카카오톡 채널 주소를 받으면features/fee/constants/fee.ts의STUDENT_COUNCIL_KAKAO_CHANNEL_URL만 채우면 된다3147:148379)은 뺐다. 신청하면 바로 확정이라 이 상태가 없다use()+ 스켈레톤이 아니다. 데이터 위치만 옮겼고, 전환은 "데이터 로딩 규칙 적용" 이슈에서 사물함·홈 등과 같이 한다⛓️ 기존 기능에 미치는 영향
LockersMapView로 옮기고 파일 이름을 바꿈. 칸 70개 모두 선택 가능, 내 칸 초록 테두리 그대로인 것을 확인했다isMine이 되어, 신청 화면에서도 A-7이 내 칸(초록 테두리, 선택 불가)으로 보인다🔀 Edge Case & 실패 시나리오
-📋 검토한 대안과 선택 이유
LockersMapView로 꺼내, 신청 화면의 확인 모달·제출 로직이 보기 화면에 섞이지 않게 했다features/my에 모으기 vs 도메인별 폴더: 데이터·컴포넌트를 도메인과 같이 두려고feedbacks·fee·lockers에 나눴다entities/feedbacks로 합쳤다💬 리뷰 포인트
[c]사물함 칸 이름 정리(LockersCell등)와LockersMapView분리 — 칸 선택 화면 동작이 그대로인지[c]피드백 데이터 이전 후 열린피드백 목록·상세·검색이 그대로인지[a]위치 보기 화면의 범례·처음 스크롤 동작