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

놀러가기 기능 #66

Merged
merged 4 commits into from
Jun 6, 2022
Merged

놀러가기 기능 #66

merged 4 commits into from
Jun 6, 2022

Conversation

litsynp
Copy link
Contributor

@litsynp litsynp commented Jun 3, 2022

DONE

친구 & 친구 요청 Entity

  • ManyToOne 관계에 대해 fetch = FetchType.LAZY 적용 참고
  • ManyToOne 관계에 대해 optional = false 적용
  • 친구를 신청한 사람과 받은 사람 필드인 from_user_idto_user_id에 대해 Unique Constraint 추가

친구 API - 놀러가기

  • FriendController에서 PUT 매핑된 메소드 이름을 update -> hangOut 으로 대체하고, UpdateServiceDto 등 Update 관련 코드를 삭제하고, 놀러가기 (hangOut)로 다시 네이밍
  • FriendQueryRepositoryhangOutOffForAllFriendsOfFromUser(UUID fromUserId) 추가
    • 주어진 사용자에 대해 놀러가기 상태인 친구 모두의 놀러가기 상태를 해제
    • Querydsl과 where의 서브쿼리(참고)를 이용해 Bulk Update (참고)
  • (Unique constraint를 대체하여) service 레벨에서 친구 모두의 놀러가기 상태를 해제시키고, 오직 한 명에 대해서만 놀러가기 상태로 만듦
    • [중요] 이 때, 친구가 아닌 다른 사용자의, 또는 친구여도 FromUserId가 다른 Friend의 놀러가기 상태에 대해서는 영향이 없도록 함
  • openapi3 업데이트
  • FriendQueryRepositoryhangOutOffForAllFriendsOfFromUser(UUID fromUserId) 테스트 코드 추가
    image

TODO

  • 놀러가기를 동시에 한 명한테만 신청할 수 있으므로 isHangingOut = true가 단 하나여야 하므로 DB 레벨에서 Unique Constraint를 추가하고 싶었으나, 애플리케이션(JPA/Hibernate) 레벨에선 안되는 것으로 보임 (어노테이션 미지원). DB에 직접 unique constaint 추가해야 적용이 가능할 것 같다.

OTHERS

  • 놀러가기를 신청하면, 신청한 친구에게 신청된 사용자가 가진 행정 구역을 조회할 수 있도록 권한이 부여된다.
    • 행정 구역 API의 구현이 우선됨. @DPS0340
  • 리뷰를 남길 때, 지도의 주인이 남긴 곳에 리뷰를 같이 남길 수 있다.
    • 리뷰 API 연동 필요 -- 리뷰 API 직접 수정 필요 @Sith-call
    • 친구 지도로 가면 친구 지도에 있는 맛집 리뷰를 볼 수 있음

Related: #62

@litsynp litsynp added the enhancement New feature or request label Jun 3, 2022
@litsynp litsynp self-assigned this Jun 3, 2022
Copy link
Member

@Sith-call Sith-call left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@himJJong
Copy link
Member

himJJong commented Jun 4, 2022

확인했습니다 고생하셨어요!!

@litsynp litsynp merged commit e178f90 into techeer-f5:main Jun 6, 2022
@litsynp litsynp deleted the feat/add-hang-out branch June 6, 2022 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

4 participants