Repository navigation
[Feat/#155] 사물함 학생앱 API 연동 - #156
Merged
Merged
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
github-actions
Bot
requested review from
jjunh33,
leegain1,
tnals0924 and
xeoxxn
November 3, 2026 17:21
…ockers-api-integration
* refactor: 빌릴게 코드 용어를 bililge에서 billilge로 변경 * docs: 용어 사전과 컨벤션 문서의 빌릴게 코드 용어를 billilge로 변경
…cture' into refactor/#153-shared-folder-structure
…ockers-api-integration
…-integration # Conflicts: # docs/conventions/coding-style.md # src/features/lockers/components/LockersAssignmentContent.tsx # src/features/lockers/components/LockersMineContent.tsx # src/shared/entities/lockers/lockersKeys.ts # src/shared/entities/lockers/lockersMock.ts # src/shared/entities/lockers/lockersQueries.ts # src/shared/entities/lockers/lockersTypes.ts
sangrae2325
changed the base branch from
refactor/#153-shared-folder-structure
to
main
November 4, 2026 10:49
This was referenced Nov 4, 2026
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.
#️⃣연관된 이슈
🎯 해결하려는 문제가 무엇인가요?
사물함 신청·배정 화면이 신청 기간·유의사항 날짜·구역 목록·칸 배치·신청 결과·내 배정을 모두 목데이터로 그렸다. dev 서버에 학생앱 사물함 API 5개가 배포돼서 실제 데이터로 연결한다.
❓ 왜 해결해야 하나요?
사물함 신청은 선착순 즉시 배정이라 남은 수·칸 상태·신청 기간이 서버 값이어야 한다. 목데이터로는 신청 기간 판정(
IS_LOCKERS_APPLY_PERIOD = true고정)과 실패 처리(목업 결과값)가 실제와 맞지 않는다.⭐ 어떻게 해결했나요?
GET /v1/app/lockers/periodOPEN일 때만 확인 버튼), 이후 모든 조회의lockerPeriodIdGET /sections?lockerPeriodIdGET /sections/{sectionId}?lockerPeriodIdPOST /applicationsGET /applications조회 정의 (
5288008) —lockersQueries에 조회 4개 + zod 응답 스키마(looseObject— 서버가 필드를 더 보내도 깨지지 않고, 빠지거나 타입이 바뀌면 여기서 실패). 타입은lockersTypes에서z.infer. zod는 이 PR에서 처음 추가(zod 4.6.7)화면 연결 (
053e04b·7ead76d·c45b210·36c249d) — 캐시에는 서버 응답 그대로 두고 화면 모양은select(toLockersNotice·toLockersSections·toLockersSectionDetail·toLockerAssignment). 여러 화면이 쓰는 변환은shared/entities/lockers에 둔다/sections/A-1)에서 서버 숫자 id(/sections/3)로 바꿨다LockersMapSkeleton이 헤더까지 채운다사진 여러 장 (
c90e524·0d36b1d) — 사진 모달 안에서 가로로 넘기고 아래에 점. 점은 로그인 소개의AuthPaginationDots를 공용components/ui/PaginationDots로 옮겨 같이 쓴다. 사진 틀은 Figma 비율(300:224)로 화면 폭에 맞춰 줄어든다(320 화면에서 카드가 넘치던 문제)신청 (
918eb74) —useApplyLockerMutation, 서버 오류 코드 → 실패 화면LOCKER_ALREADY_ASSIGNEDMEMBER_ALREADY_HAS_LOCKERLOCKER_APPLY_NOT_OPEN·LOCKER_PERIOD_NOT_FOUND점검 반영 (
8a63257·3e1c2fd·5480d48)정리·문서 (
0e7ff35) — 칸 배치·배정 목데이터와 구역 사진 jpg 삭제,wds-component-usage.md·coding-style.md갱신🧩 이 PR의 한계 & 트레이드오프
LockerSectionDetailResponse등)의 응답 모양 그대로 만든 응답을 브라우저에서 가로채 확인했다homeMock— 홈 API를 붙일 때 바꾼다formatDateTime은 일시에T가 없으면 던진다 — #151과 같은 줄이라 [Feat/#149] 학생회비 납부 API 연결 #151 머지 뒤 따로 고친다⛓️ 기존 기능에 미치는 영향
components/ui/PaginationDots이동으로 로그인 소개 화면(AuthLoginCarousel) import가 바뀐다(동작 같음)lib/date의formatDateTime이 시:분까지만 남긴다(서버 일시가 초까지 온다) — #151과 같은 줄로 맞춰 충돌하지 않는다/sections/A-1)는 준비 중 화면이 된다🔀 Edge Case & 실패 시나리오
375×812에서 확인(서버 응답 가로채기):
OPEN·BEFORE_OPEN·CLOSED·회차 없음version·형식 틀린 블록 → 빈 화면, 선택 불가 칸 토스트📋 검토한 대안과 선택 이유
queryFn안에서 — 같은 응답을 여러 화면이 다른 모양으로 써서(내 배정 → 배정 상태·위치 보기) 캐시는 응답 그대로, 변환은select([Refactor/#153] API·도메인 조회 코드를 shared 폴더 구조로 정리 #154 컨벤션)💬 리뷰 포인트
[c]Figma에 없는 화면 문구 — 이미 배정됨 실패 화면("이미 사물함을 배정받았어요 / 배정된 사물함을 확인해 보세요."), 칸 배치 빈 화면("아직 칸 배치를 볼 수 없어요 / 다른 구역을 선택해 주세요")[c]shared/entities/lockers/lockersQueries.ts— zod 스키마·select변환·mutation 캐시 갱신[a]사진 모달 점(PaginationDots)이 Figma와 같은 모양인지