feat: 주문 상점 전환 POST API를 추가하다 #2049

asa9874 · 2025-10-27T15:37:22Z

🔍 개요

주문 상점 전환 POST API를 추가하다

close [비즈니스] 주문 가능 상점 전환(주문 서비스 가입) POST API 추가 #2027

🚀 주요 변경 내용

주문 상점 전환 POST API 추가
테스트 코드 포함
FLYWAY CREATE TABLE 포함
상점관의 관계 ManyToOne으로 생성 (반려후에 다시 신청할수있도록)
- 이렇게 바꾸니 동시 요청에 대한 방어가 안될거같네요

💬 참고 사항

accountNumber 평문 저장? (개인정보?)
상점관의 관계 ManyToOne관련 이슈?
- 이렇게 바꾸니 동시 요청에 대한 방어가 안될거같네요 .....

✅ Checklist (완료 조건)

코드 스타일 가이드 준수
테스트 코드 포함됨
Reviewers / Assignees / Labels 지정 완료
보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

* feat: 빈 java 파일들 생성 * feat: ShopToOrderable Post를 스켈레톤코드 작성 * refact: DTO, Model 필드 생성 (임시) * feat: Controller 수정 * feat: Service, Repository 단 * refact: 컨벤션 수정 * refact: 예외코드 추가 및 수정, 엔드포인트 경로 변경, * refact: Entity 리펙토링 * refact: 미사용 import 제거 * refact: {shopId} 추가 및 개행 제거

* feat: 검증 추가 * feat: 주문가능상점변환 테스트 추가 * refact: 매개변수 stirng -> ShopToOrderableRequestStatus로 변경 * refact: 예외 명 변경

github-actions · 2025-10-27T15:41:12Z

Unit Test Results

673 tests 672 ✔️ 1m 14s ⏱️
165 suites     1 💤
165 files     0 ❌

Results for commit 7755783.

♻️ This comment has been updated with latest results.

Soundbar91

고생하셨습니다.
지금 돌아가는 건 확인하지 못해서, 코드 보고 확인할 수 있는 부분에 대해서만 코멘트 달았습니다.
확인 부탁드립니다 !

accountNumber 평문 저장? (개인정보?)

이게 계좌번호인거죠 ? 정리된 글을 보니까, 암호화해야할 거 같아요.
슬랙에서 한 번 이야기 나눠보면 좋을 거 같아욥

상점관의 관계 ManyToOne관련 이슈

~OneToOne에서 ManyToOne으로 바꾸신 이유가 궁금합니다 👀 ~
-> 아 적혀있었네요..! 동시 요청 관련해서 어노테이션이 있어서, 한 번 적용해보는 것도 좋을 거 같아요.

Soundbar91 · 2025-10-28T08:00:55Z

...main/java/in/koreatech/koin/domain/shoptoOrderable/controller/ShopToOrderableController.java

+import in.koreatech.koin.global.auth.Auth;
+import jakarta.validation.Valid;
+
+//Todo: ShopToOrderable 이라는 명칭은 임시임 추후 변경


C

ShopToOrderable으로 확정하신건가용
개인적으로는 ownershop 패키지 하위로 들어가도 괜찮을 거 같은데 어떻게 생각하시나요 ??

해당 스프린트가 아직 명확하게 요구사항이 정해지지 않아서, 우선 독립적으로 패키지를 만들어 제작을 시작했습니다.
추후 요구사항이 명확해지게 되면, 프로젝트 구조에 맞게 패키지 구조를 조정할려고 하는데 괜찮을까요?

src/main/java/in/koreatech/koin/domain/shoptoOrderable/dto/ShopToOrderableRequest.java

src/main/java/in/koreatech/koin/domain/shoptoOrderable/model/ShopToOrderable.java

Soundbar91 · 2025-10-28T08:13:39Z

src/main/java/in/koreatech/koin/domain/shoptoOrderable/model/ShopToOrderable.java

+    @Size(max = 10)
+    @Column(name = "bank", length = 10, nullable = false)
+    private String bank;


A

bank를 String 말고 enum으로 관리할 수 있을까요 👀
db에서 관리하기 ?

해당 부분은 요구사항측에서 은행의 종류같은게 정해지면 enum으로 관리 하는게 좋을거같아요.

db에서 관리는 enum 말고 테이블로 (1, "OO은행") , (2,"AA은행")으로해서 연관으로 관리한다는 의미인가요?

Soundbar91 · 2025-10-28T08:14:46Z

src/main/java/in/koreatech/koin/domain/shoptoOrderable/model/ShopToOrderable.java

+    private String accountNumber;
+
+    @Enumerated(EnumType.STRING)
+    @Column(name = "request_status", nullable = false)


A

status으로 표현해도 괜찮을 거 같아요

src/main/java/in/koreatech/koin/domain/shoptoOrderable/model/ShopToOrderable.java

...main/java/in/koreatech/koin/domain/shoptoOrderable/repository/ShopToOrderableRepository.java

src/main/java/in/koreatech/koin/domain/shoptoOrderable/service/ShopToOrderableService.java

Soundbar91

확인해주세욥

src/main/resources/db/migration/V226__add_shop_to_orderable_request.sql

src/main/java/in/koreatech/koin/domain/shoptoOrderable/service/ShopToOrderableService.java

Soundbar91

SwaggerGroupConfig 클래스에서 스웨거 그룹을 설정할 수 있습니다 !
패키지를 새롭게 추가하는 경우 패키지 경로를 넣어주셔야, 스웨거에서 확인이 가능합니당

@Bean
    public GroupedOpenApi businessApi() {
        return createGroupedOpenApi(
            "2. Business API",
            new String[] {
                "in.koreatech.koin.domain.owner",
                "in.koreatech.koin.domain.benefit",
                "in.koreatech.koin.domain.ownershop",
                "in.koreatech.koin.domain.shop",
                "in.koreatech.koin.domain.land",
                "in.koreatech.koin.domain.shoptoOrderable",
            });
    }

asa9874 · 2025-10-28T12:28:02Z

@DuplicateGuard 추가로 동시성 방지를 추가했습니다

Swagger 에서의 테스트

1. 자기 상점이 아닌경우

2. 자기 상점인 경우

3. PENDING 상태인 요청이 있는경우

4. APPROVED 상태인 요청이 있는경우

5. REJECTED 상태인 요청이 있는경우 (PENDING , APPROVED 없는경우 )

asa9874 added 4 commits October 13, 2025 11:25

feat : flyway 추가 및 컬럼 명 추가

1c2c021

refact : isOpen 삭제

b93a413

asa9874 self-assigned this Oct 27, 2025

asa9874 added 기능 새로운 기능을 개발합니다. Team Business 비즈니스 팀에서 작업할 이슈입니다 labels Oct 27, 2025

github-actions bot requested review from DHkimgit, ImTotem and Soundbar91 October 27, 2025 15:37

asa9874 changed the title ~~feat : 주문 상점 전환 POST API를 추가하다~~ feat: 주문 상점 전환 POST API를 추가하다 Oct 27, 2025

asa9874 added 7 commits October 28, 2025 00:43

fix : isOpen 테스트 부분 삭제

4c79e3c

refact : idx제거

a5e9b71

refact : 형식 준수

813e8d7

feat : fk_shop_to_orderable_shop 외래키 추가

e8286ba

fix : approvedAt NULL 불일치 제거

e2613fc

feat: 상점 @manytoone으로 변경

c713291

feat: ShopToOrderableDeliveryOption 추가

a45169e

Soundbar91 reviewed Oct 28, 2025

View reviewed changes

refact: @NotNull 추가

b67f462

Soundbar91 requested changes Oct 28, 2025

View reviewed changes

src/main/resources/db/migration/V226__add_shop_to_orderable_request.sql Outdated Show resolved Hide resolved

src/main/java/in/koreatech/koin/domain/shoptoOrderable/service/ShopToOrderableService.java Outdated Show resolved Hide resolved

refact: 검증로직 메서드화

dee35e5

Soundbar91 reviewed Oct 28, 2025

View reviewed changes

asa9874 added 6 commits October 28, 2025 20:14

refact: 개행제거

ad017d3

fix : flyway fk 테이블명 불일치 해결

697929e

feat: Swagger 패키지 추가

769d069

refact: takeout -> isTakeout 컬럼명 수정

14d3471

refact: outside_delivery_tip -> off_campus_delivery_tip 컬럼명 수정

67d540c

feat : 동시성 방지 @DuplicateGuard 추가

683f121

refact : 가게 사장 검증로직 shop 내부 메서드로 이동

7755783

asa9874 requested a review from Soundbar91 October 28, 2025 12:28

Uh oh!

feat: 주문 상점 전환 POST API를 추가하다 #2049

Are you sure you want to change the base?

feat: 주문 상점 전환 POST API를 추가하다 #2049

Conversation

asa9874 commented Oct 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔍 개요

🚀 주요 변경 내용

💬 참고 사항

✅ Checklist (완료 조건)

Uh oh!

github-actions bot commented Oct 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Unit Test Results

Uh oh!

Soundbar91 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Soundbar91 Oct 28, 2025

Choose a reason for hiding this comment

C

Uh oh!

asa9874 Oct 28, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Soundbar91 Oct 28, 2025

Choose a reason for hiding this comment

A

Uh oh!

asa9874 Oct 28, 2025

Choose a reason for hiding this comment

Uh oh!

Soundbar91 Oct 28, 2025

Choose a reason for hiding this comment

A

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Soundbar91 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Soundbar91 left a comment

Choose a reason for hiding this comment

Uh oh!

asa9874 commented Oct 28, 2025

Swagger 에서의 테스트

1. 자기 상점이 아닌경우

2. 자기 상점인 경우

3. PENDING 상태인 요청이 있는경우

4. APPROVED 상태인 요청이 있는경우

5. REJECTED 상태인 요청이 있는경우 (PENDING , APPROVED 없는경우 )

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

asa9874 commented Oct 27, 2025 •

edited

Loading

github-actions bot commented Oct 27, 2025 •

edited

Loading

Soundbar91 left a comment •

edited

Loading