Skip to content

[REFACTOR/#453] store 조회에 link 리다이렉트 여부 판별 필드 추가 - #454

Merged
eeeeeaaan merged 1 commit into
developfrom
refactor/#453-store-paper-category
Sep 6, 2026
Merged

[REFACTOR/#453] store 조회에 link 리다이렉트 여부 판별 필드 추가#454
eeeeeaaan merged 1 commit into
developfrom
refactor/#453-store-paper-category

Conversation

@eeeeeaaan

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

close #453

📝작업 내용

가게(Store)가 외부 링크로 리다이렉트되어야 하는 대상인지 구분하기 위해 linkType enum 필드를 추가했습니다.
실제 리다이렉트 URL은 프론트에서 처리하므로, 서버는 리다이렉트 대상 여부/타입만 내려줍니다. 해당 없는 일반 가게는 null입니다.

🔎코드 설명(스크린샷(선택))

  • Store 엔티티에 linkType 필드 추가 (@Enumerated(EnumType.STRING), nullable)
  • domain/store/entity/enums/LinkType.java 신규 생성 (현재 EXTERNAL 단일 값)
  • GET /store/{storeId} (가게 상세 조회) — GetStoreDetailsDTOlinkType 추가, StoreServiceImpl.getStoreDetails에서 값 채움
  • GET /store/{storeId}/papers (제휴 컨텐츠 조회) — PaperResponseDTOlinkType 추가, PaperQueryServiceImpl에서 값 채움
  • GET /map/nearby, GET /map/search (STUDENT 응답) — StoreMapResponseDTOlinkType 추가, ofWithPartnerships에서 값 채움
  • 3개 API의 @Operation Swagger 설명에 linkType 필드 설명 추가
  • AdminMapResponseDTO, PartnerMapResponseDTO는 Store가 아닌 Admin/Partner 엔티티 기반이라 이번 변경에서 제외

💬고민사항 및 리뷰 요구사항 (Optional)

  • ddl-auto: update 설정으로 컬럼은 자동 반영되나, prod 배포 전 실제 반영 여부 확인이 필요합니다.
  • 현재 linkType 값을 등록/수정하는 API(백오피스/파트너)는 없습니다. Store 생성 시(SignUpServiceImpl) 기본값 null로만 들어가며, 값 세팅 경로는 후속 작업으로 남겨두었습니다.
  • 기존 필드는 변경 없이 필드 추가만 이루어져 하위 호환에는 영향 없습니다.

비고 (Optional)

프론트 공유 메시지: 3개 API(GET /store/{storeId}, GET /store/{storeId}/papers, GET /map/nearby, GET /map/search)에 linkType 필드가 추가되었으며, null이면 일반 가게로 처리하면 됩니다.

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: b6600adc-06fc-4f37-a071-209b1afc18e1


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.

@eeeeeaaan eeeeeaaan self-assigned this Sep 5, 2026
@leesumin0526

Copy link
Copy Markdown
Contributor

이넘 처리 좋습니다~ 감사합니다!

@2ghrms 2ghrms left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

지도에서 표현되어야할 가게인지, 외부 링크로 표현해야할 가게인지를 나타내는 게 목적이니 제 생각엔 LinkType보다는 가게타입 혹은 제휴타입 이런 변수명을 사용하고 MAP, EXTERNAL_LINK 이런식으로 Enum 처리하는게 좋을 것 같아여

@eeeeeaaan

eeeeeaaan commented Sep 6, 2026

Copy link
Copy Markdown
Contributor Author

지도에서 표현되어야할 가게인지, 외부 링크로 표현해야할 가게인지를 나타내는 게 목적이니 제 생각엔 LinkType보다는 가게타입 혹은 제휴타입 이런 변수명을 사용하고 MAP, EXTERNAL_LINK 이런식으로 Enum 처리하는게 좋을 것 같아여

제가 이부분에 대해서 생각을 해봤는데, 이게 제휴 타입으로 가는게 사실상 맞는데 지도에서 리다이렉트를 하다보니 store에 필드가 붙게 되고 그래서 스토어 타입이라고 짓기엔 적합하지 않다... 라고 생각했습니다. 그렇다고 .store 테이블에 제휴 타입이라는 필드를 붙이는 것도 아닌 것 같아서 애매하게 처리하게 된 것 같습니다

@eeeeeaaan
eeeeeaaan merged commit 32bffd4 into develop Sep 6, 2026
2 checks passed
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.

[REFACTOR/#453] store 테이블에 제휴 카테고리 추가

3 participants