Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BE] feat: 반려견 정보 수정 API #398

Merged
merged 5 commits into from
Aug 14, 2024
Merged

[BE] feat: 반려견 정보 수정 API #398

merged 5 commits into from
Aug 14, 2024

Conversation

takoyakimchi
Copy link
Contributor

@takoyakimchi takoyakimchi commented Aug 13, 2024 •
edited
Loading

이슈

개발 사항

  • 반려견 정보 수정 API 구현 및 문서화

리뷰 요청 사항

  • update 메서드 + 변경감지로 수정 구현했습니다!

전달 사항

  • API 수정은 없고 추가만 되었습니다. 바로 merge해도 됩니다.
  • image가 null인 경우 이미지 수정 없이 기존 이미지를 사용합니다.
  • image가 null이 아닌 경우, 기존 이미지를 새로운 이미지로 대체합니다.
  • multipart에 대한 API 문서화를 할 방법이 없어 summary에 String으로 작성하였습니다.

@takoyakimchi takoyakimchi added 🖥 backend backend ✨ feat 기능 개발 labels Aug 13, 2024
@takoyakimchi takoyakimchi added this to the Sprint4 milestone Aug 13, 2024
@takoyakimchi takoyakimchi self-assigned this Aug 13, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 13, 2024 •
edited
Loading

Test Results

133 tests  +130   133 ✅ +130   16s ⏱️ +16s
 27 suites + 26     0 💤 ±  0 
 27 files   + 26     0 ❌ ±  0 

Results for commit 2054907. ± Comparison against base commit 3f24d61.

This pull request removes 3 and adds 133 tests. Note that renamed tests count towards both. 
com.happy.friendogly.ExampleUnitTest ‑ addition_isCorrect
com.happy.friendogly.ExampleUnitTest ‑ addition_isCorrect2
com.happy.friendogly.ExampleUnitTest ‑ addition_isCorrect3

com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 1대1 채팅방 저장 과정에서, 이미 채팅방이 존재하면 기존 채팅방 ID를 반환한다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 모임에 참여하지 않은 경우, 모임 채팅방에서 나갈 수 없다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 새로운 1대1 채팅방을 개설할 수 있다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 채팅방에 참여한 경우, 채팅방에서 나갈 수 있다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 내가 속해 있는 채팅방을 찾을 수 있다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 자신이 참여한 채팅방이 아니면 채팅방에서 모임 정보를 받아올 수 없다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 채팅방 ID로부터 모임 ID, 허용 펫 사이즈, 허용 펫 성별을 조회할 수 있다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 채팅방 내 멤버 세부정보를 조회할 수 있다.
com.happy.friendogly.club.domain.AddressTest ‑ [1] provinceInput=null
com.happy.friendogly.club.domain.AddressTest ‑ [2] provinceInput=
…

♻️ This comment has been updated with latest results.

J-I-H-O
J-I-H-O approved these changes Aug 14, 2024
View reviewed changes
Copy link
Contributor

@J-I-H-O J-I-H-O left a comment

Choose a reason for hiding this comment

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

고생했습니다!

jimi567
jimi567 approved these changes Aug 14, 2024
View reviewed changes
Copy link
Member

@jimi567 jimi567 left a comment

Choose a reason for hiding this comment

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

모임에 참여한 상태에서 갑자기 젠더 타입이나 사이즈가 바뀌면 곤란한 상황이 생길 수 있을 것 같은데 이거는 일단 넘어갑시다..

backend/src/main/java/com/happy/friendogly/pet/service/PetCommandService.java
Comment on lines +73 to +76
if (!pet.isOwner(member)) {
throw new FriendoglyException("자신의 강아지만 수정할 수 있습니다.");
}

Copy link
Member

Choose a reason for hiding this comment

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

이건 인가에 관련한 내용이라 403 등으로 분류해도 좋을 것 같아요.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

모임에 참여한 상태에서 갑자기 젠더 타입이나 사이즈가 바뀌면 곤란한 상황이 생길 수 있을 것 같은데 이거는 일단 넘어갑시다..

이건 따로 issue 파야겠네요... ㄷㄷㄷ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이건 인가에 관련한 내용이라 403 등으로 분류해도 좋을 것 같아요.

S3 이미지 삭제 구현하면서 반영해볼게요~

@J-I-H-O J-I-H-O merged commit e1b3fa0 into develop Aug 14, 2024
3 checks passed
@J-I-H-O J-I-H-O deleted the feature/be/#371 branch August 14, 2024 07:33
takoyakimchi added a commit that referenced this pull request Oct 23, 2024
@takoyakimchi
[BE] feat: 반려견 정보 수정 API (#398)
6e02f15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@J-I-H-O J-I-H-O J-I-H-O approved these changes

@ehtjsv2 ehtjsv2 ehtjsv2 approved these changes

@jimi567 jimi567 jimi567 approved these changes

Assignees

@takoyakimchi takoyakimchi

Labels
🖥 backend backend ✨ feat 기능 개발
Projects
2024-friendogly
Status: Done
Milestone
Sprint4
Development

Successfully merging this pull request may close these issues.

강아지 정보 수정 API
4 participants
@takoyakimchi @jimi567 @ehtjsv2 @J-I-H-O