Skip to content

feat: [ALT-321] 업장 주소를 기본·지번·상세 주소로 나눠 저장 - #122

Draft
ysw789 wants to merge 2 commits into
devfrom
feat/alt-321
Draft

ysw789 wants to merge 2 commits into
devfrom
feat/alt-321

Conversation

@ysw789

@ysw789 ysw789 commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

변경사항

https://app.notion.com/p/4078655316288194a481fd6dc66d7181

업장 등록 신청 때 주소를 기본 주소(도로명)·지번 주소·상세 주소(동·층·호수)로 나눠 받습니다. 클라이언트가 좌표를 조회할 때 기본 주소만 쓰고, 상세 주소는 다음 화면에서 따로 입력하도록 하기 위한 변경입니다 (alter-client#69 후속).

  • workspace_requests, workspaces에 jibun_address(text), detail_address(varchar(128)) 컬럼 추가. 둘 다 NULL 허용이고 기존 행은 NULL로 남음 (V18 마이그레이션)
  • 등록 신청 요청에 jibunAddress, detailAddress(최대 128자) 추가. 둘 다 선택 입력
  • 신청을 승인하면 세 주소를 업장으로 그대로 복사
  • 신청 목록·상세(앱·사장님·관리자), 사장님 업장 목록·상세, 공고 상세의 업장 정보 응답에 jibunAddress, detailAddress를 따로 내려줌
  • 기존 fullAddress 컬럼과 필드명은 그대로 두고 의미만 "기본 주소(도로명)"로 정리

참고

  • 마이그레이션 번호: V17은 아직 병합되지 않은 feat/alt-309가 쓰고 있어 V18로 만들었습니다. out-of-order: false 설정이라 이 PR이 먼저 병합되면 V17로 바꿔야 합니다.
  • 테스트: 등록·승인 UseCase 단위 테스트를 추가했고 ./gradlew clean build 전체 통과했습니다.
  • 로컬 API 테스트
    • 신청: 지번·상세 주소를 넣은 경우와 뺀 경우 모두 200, 상세 주소 128자는 200·129자는 400/B001
    • 조회: 신청 목록·상세, 승인 후 업장 목록·상세, 공고 상세에서 세 주소가 따로 내려오는 것을 확인했습니다.

@coderabbitai

coderabbitai Bot commented Oct 28, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@ysw789 ysw789 self-assigned this Oct 28, 2026

This branch has not been deployed

No deployments
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.

1 participant