Skip to content

#138 - 쪽지 답장을 사용한적이 있는지 확인하는 API를 구현합니다. #163

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 6, 2025

Conversation

kpeel5839
Copy link
Contributor

1. 🔗 관련 이슈

closes #138

2. 📄 구현한 내용 또는 수정한 내용

  • 쪽지 답장을 사용한적이 있는지 확인하는 API를 구현했습니다.

3. ✅ 배포 Checklist

  • 본인을 Assign 해주세요.
  • 본인을 제외한 백엔드 개발자를 리뷰어로 지정해주세요.
  • 변경된 DB 업데이트 해주세요. (꼭 해주세요 배포 안돼요!!!)

@kpeel5839 kpeel5839 requested a review from Copilot May 6, 2025 08:25
@kpeel5839 kpeel5839 self-assigned this May 6, 2025
@kpeel5839 kpeel5839 added 🌱기능🌱 새로운 기능을 추가해요 ! 🏋️매튜🏋️ 24기 김재연 labels May 6, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements an API endpoint to check whether a user has previously used the answer message feature.

  • Adds a new repository method to retrieve answer message usage by user ID.
  • Implements a persistence adapter and service method for domain mapping.
  • Introduces a REST controller to expose the feature status via a GET endpoint.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
infrastructure/mysql/src/main/kotlin/com/wespot/user/repository/UsedAnswerMessageJpaRepository.kt Added repository method findByUserId to support domain lookup.
infrastructure/mysql/src/main/kotlin/com/wespot/user/adapter/UsedAnswerMessagePersistenceAdapter.kt Mapped the repository entity to the domain model via a new method.
core/src/main/kotlin/com/wespot/user/service/UsedAnswerMessageService.kt Created a service that retrieves the feature status for a user.
core/src/main/kotlin/com/wespot/user/port/in/UsedAnswerMessageUseCase.kt Defined a new use case interface for the feature check.
app/src/main/kotlin/com/wespot/user/UsedAnswerMessageController.kt Added a controller exposing the API endpoint for the feature check.



@RestController
@RequestMapping("/api/v2/messages/answer/first")
Copy link
Preview

Copilot AI May 6, 2025

Choose a reason for hiding this comment

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

[nitpick] The endpoint path '/api/v2/messages/answer/first' does not clearly indicate that it checks for the usage of the answer message feature. Consider renaming the path to something more descriptive, such as '/api/v2/messages/answer/feature-status', to better convey its purpose.

Suggested change
@RequestMapping("/api/v2/messages/answer/first")
@RequestMapping("/api/v2/messages/answer/feature-status")

Copilot uses AI. Check for mistakes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No Thanks

@kpeel5839 kpeel5839 merged commit aa2d7e9 into develop May 6, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱기능🌱 새로운 기능을 추가해요 ! 🏋️매튜🏋️ 24기 김재연
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature]: 이전에 답장하기를 해본적이 있는지에 대한 여부
1 participant