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.

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

#62 [FEAT, FIX] Feature/edit group 그룹 api 수정 및 개발 #72

Merged
merged 8 commits into from
Nov 4, 2023

Conversation

hye-on
Copy link
Contributor

@hye-on hye-on commented Nov 2, 2023

☀️ 작업 사항

  • room get api에 초대코드 추가
  • 초대받은 사람이 보는 room 정보 api 개발
  • roomUser null 체크 Optional 클래스에 위임하게 수정
  • 자잘한 수정
    • 주석삭제
    • 아바타 못찾을때 상태코드 수정
    • dml 수정

☀️ 관련 이슈

#62

☀️ 참고 사항

  • erd 변경되었습니다. room 초대코드 길이 수정

Copy link
Member

@05AM 05AM left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Comment on lines 206 to 207
RoomUser seletedRoomUser = roomUserRepository.findByRoom_IdAndUser_Id(roomId, selectedUserId)
.orElseThrow();
Copy link
Member

Choose a reason for hiding this comment

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

P3: 아무것도 throw하지 않은 이유가 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

놓친것같습니다! 감사합니다

@@ -101,7 +102,7 @@ public BaseResponse changeRoomHost(@Parameter(hidden = true) @AuthUser Integer u

}

@Operation(summary = "그룹 디테일 보기 and 초대 받은 사람에게 보이는 그룹 정보 보기 ",description = "그룹 디테일 보기 (설정 화면) and 초대 받은 사람에게 보이는 앱 내 페이지")
@Operation(summary = "그룹 디테일 보기 ",description = "그룹 디테일 보기 (설정 화면)")
Copy link
Member

Choose a reason for hiding this comment

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

P3: 혹시 코드 컨벤션 적용하셨을까요? 군데군데 적용이 되지 않은 부분이 보이는 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

어떤 코드 컨벤션 말하는 걸까요?
뛰어쓰기 말씀하신건가요?

Copy link
Member

Choose a reason for hiding this comment

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

인텔리제이에서 컨트롤 알트 L을 누르면 자동으로 포맷팅 되는 기능이 있습니다! 그걸 적용하셨는지 궁금합니다!
아니면 Save and action에서 저장할 때마다 코드 포맷팅을 하는 기능도 있습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

글 정렬은 따로 pr을 올리는 것이 좋을 것같아서 따로 pr을 올렸다가 이 pr이 머지 된후에 다시 올리려고 close 하였습니다.
다시 알려주셔서 감사합니다😃

@hye-on hye-on merged commit e06809a into main Nov 4, 2023
1 check 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.

2 participants