Skip to content

Implementing CRUD APIs for Business Inquiries - #412

Merged
Whale0928 merged 8 commits into
devfrom
feat/406
Jul 2, 2025
Merged

Implementing CRUD APIs for Business Inquiries#412
Whale0928 merged 8 commits into
devfrom
feat/406

Conversation

@chadongmin

Copy link
Copy Markdown
Contributor

Resolves #{이슈-번호}

https://github.com/bottle-note/bottle-note-api-server/issues/406#event-18229260893

해결하려는 문제가 무엇인가요?

  • 비지니스 문의 API 구현

RCA 룰

r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

@chadongmin chadongmin self-assigned this Jun 26, 2025
@chadongmin chadongmin linked an issue Jun 26, 2025 that may be closed by this pull request
13 tasks
@chadongmin
chadongmin requested a review from Whale0928 June 26, 2025 11:32
… 생성하도록 수정

- 변경 불가능한 리스트를 직접 수정해서 발생하던 UnsupportedOperationException을 해결
Whale0928
Whale0928 previously approved these changes Jun 26, 2025

@Whale0928 Whale0928 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

전체적으로 준수하게 구현되엇는대 관습을 좀더 맞추죠
특정한 형태가 일관성있게 유지된다면
Claude code 사용중이면 코드 베이스를 탐색해서 기존 타입 매칭을 하는것도 좋습니다.

Comment thread src/main/java/app/bottlenote/support/business/service/BusinessSupportService.java Outdated
Comment thread src/main/java/app/bottlenote/support/business/service/BusinessSupportService.java Outdated
@Whale0928
Whale0928 self-requested a review June 26, 2025 13:54
- MockTest -> FakeRepositoryTest로 수정
- CollectionResponse객체 적용
- 적절한 예외를 던지도록 수정
@chadongmin

chadongmin commented Jun 29, 2025

Copy link
Copy Markdown
Contributor Author

@Whale0928 집에 물새는 이슈가 있어서 수리하느라 PR제출에 좀 늦었습니다 ㅜㅜ
리뷰 반영해서 푸쉬했습니다 !!

@Whale0928 Whale0928 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good

@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2025

Copy link
Copy Markdown

@Whale0928
Whale0928 merged commit 2d504cb into dev Jul 2, 2025
3 checks passed
@Whale0928
Whale0928 deleted the feat/406 branch July 2, 2025 14:46
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.

비지니스 문의 API 기능 구현

3 participants