✨ Feat: 동승 파티 참여 신청 수락/취소/거절/내보내기 기능 추가 - [merged] #26

Closed GATBAWI closed 4 months ago

In GitLab by @ajeng518 on May 5, 2024, 01:08

Merges BE/feat/party-join-accept -> develop

✨ Feat: 동승 파티 참여 신청 수락

✍ 작업 내용 [FE/BE]

동승 파티 참여 신청을 파티장만 수락 가능하도록 함
파티장만 동승 파티 참여 신청자 혹은 참여자를 거절 할 수 있도록 설정
반환 값으로 신청자 및 참여자 구분

MR 타입

[x] 기능 추가
[x] 기능 수정
[ ] 기능 삭제
[ ] 버그 수정
[ ] 코드 리팩토링
[ ] UI 추가 및 변경

반영 브랜치

BE/feat/party-join-accept -> develop

테스트 결과

[x] Swagger 테스트 결과 이상 없습니다.
[ ] local ci 테스트 결과 이상 없습니다.
[ ] Postman 테스트 결과 이상 없습니다.

In GitLab by @ajeng518 on May 5, 2024, 01:08

requested review from @soomni20

In GitLab by @tnatna0801 on May 5, 2024, 01:12

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/controller/PartyJoinController.java line 15

오호

In GitLab by @tnatna0801 on May 5, 2024, 01:13

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/controller/PartyJoinController.java line 23

이거 근데 id를 반환하는게 의미가 있나요?

In GitLab by @tnatna0801 on May 5, 2024, 01:15

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/respository/custom/CustomPartyMember.java line 26

메소드 명에 parti와 Req은 줄임말인가요?

In GitLab by @tnatna0801 on May 5, 2024, 01:16

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 113

getParty 메서드를 따로 만든 이유는 뭔가용?

In GitLab by @tnatna0801 on May 5, 2024, 01:18

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 195

여기서는 그 아정씨가 만든 트랜잭션 핸들러를 안쓴 이유가 뭔가용? 트랜잭션 핸들러는 어떤 경우에만 쓰는 건가요?

In GitLab by @tnatna0801 on May 5, 2024, 01:19

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

역할을 반환하는 이유가 궁금합니다.

In GitLab by @ajeng518 on May 5, 2024, 01:21

added 1 commit

b3949f64 - 🐛 Fix: 동승 파티 신청 시 파티상태 및 인원 체크 처리 수정

Compare with previous version

In GitLab by @ajeng518 on May 5, 2024, 01:22

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/controller/PartyJoinController.java line 15

파티 아이디 부분까지 모든 url이 중복되어서 최상단에 올려봤습니다... 혹시 저런 고유 id의 경우는 매소드명위에 많이 하나여??

In GitLab by @ajeng518 on May 5, 2024, 01:24

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/controller/PartyJoinController.java line 23

의미 없습니다.. 그냥 이게 수락의 성공 했다는 거만 알려주면 되는거라서 일단 아이디 값을 줘 봤습니당 다른 부분은 boolean으로 줬어서 통일 할지 혹시 다른방법으로 바꿔야 할지 고민입니당

In GitLab by @ajeng518 on May 5, 2024, 01:25

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/respository/custom/CustomPartyMember.java line 26

네 맞습니다. 저거 처음에 그냥 findPartyMemberByMember로 사용했는데 저게 member가 신청자와 참여자값일 경우만 가져와서 (파티장은 조건에 없음) 이름만 보면 직관적인것 같지 않아서 저렇게 바꿨습니다

In GitLab by @tnatna0801 on May 5, 2024, 01:25

approved this merge request

In GitLab by @tnatna0801 on May 5, 2024, 01:26

멋집니다 여아정씨 고생하셨어용

In GitLab by @tnatna0801 on May 5, 2024, 01:26

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/controller/PartyJoinController.java line 15

잘몰라용 우하하

In GitLab by @ajeng518 on May 5, 2024, 01:26

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 113

저 getParty는 이수민씨 께서 만들었는데 같은 파일이라 충돌을 해결하면서 getParty 메서드를 예외처리랑 다 되어있길래 적용해 보았습니당

In GitLab by @tnatna0801 on May 5, 2024, 01:27

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/controller/PartyJoinController.java line 23

그냥 void해줘도 될 것 같기두? 아니면 party id를 반환해도 될 것 같기두?

In GitLab by @tnatna0801 on May 5, 2024, 01:27

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/respository/custom/CustomPartyMember.java line 26

먼가 나중에 메서드 명도 통일하고 싶네유

In GitLab by @yaj518 on May 5, 2024, 01:28

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 195

사실 쓸지 고민했는데 update 부분은 일단 안 썼습니다. 저도 lock 걸 곳과 아닌 곳을 더 구분해서 적용해보려 합니당..! 더 알아보고 알려드리겠습니다!!

In GitLab by @tnatna0801 on May 5, 2024, 01:28

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 113

아하 흠 하나로 합쳐야겠는데요 일단 저는 findParty를 Optional로 바꿔서 곧 올릴 거긴 합니당 나중에 리팩토링 하시죠

In GitLab by @yaj518 on May 5, 2024, 01:29

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

이 부분의 경우 처음에 api 명세서에 보면 참여자를 내보내기 하는 것, 신청자를 거절 하는 것으로 두 개가 나눠져있는데 제 생각에 로직은 delete 하는걸로 크게 다르지 않을 것 같다는 생각이 들어서 하나의 로직으로 하되, 해당 사용자의 role을 구분해서 보여주는 방식을 사용해 보았습니다

In GitLab by @ajeng518 on May 5, 2024, 01:29

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/controller/PartyJoinController.java line 23

엇? void 되는 거였어? 그러면 바로해야징 ㅎ히ㅣㅎ

In GitLab by @ajeng518 on May 5, 2024, 01:30

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/respository/custom/CustomPartyMember.java line 26

네 좋습니다 다 맞춥시다!!

In GitLab by @yaj518 on May 5, 2024, 01:30

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 113

좋습니다 정리 함 해야겠군용

In GitLab by @tnatna0801 on May 5, 2024, 01:31

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

오호 동적 쿼리로 해보실 생각 있나요?

In GitLab by @tnatna0801 on May 5, 2024, 01:31

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

근데 그럼 프론트가 role을 구분해서 받아서 어떤 처리를 하나요?

In GitLab by @ajeng518 on May 5, 2024, 01:32

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

헉 동적쿼리 해보고 싶습니다. 적용해볼게용

In GitLab by @ajeng518 on May 5, 2024, 01:33

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

프론트도 아마 신청자와 참여자가 보이는 버튼이 다를 것 같아 크게 해당 role로 처리하는게 있을 것 같진 않습니다

In GitLab by @tnatna0801 on May 5, 2024, 01:41

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

이해가 쬐끔 안갑니다. 이 메소드 전체 로직이 이게 맞나요?

참여자인지 신청자인지 확인해서
참여자면 내보내고
신청자면 거절한다?
내보낸 사람이 어떤 역할이었는지 알려준다

이게 맞나요? 맞으면 일단 메소드에 findPartyPartiAndReqByMember 여기서 AND가 아니라 OR이어야 직관적일 것 같네요 그리고 맞다면 4번이 더 이해가 안가용 만약에 신청자와 참여자가 보이는 버튼이 다르다면 이 api를 요청할 때 이미 그게 판단이 되어서 처리를 했을 텐데 처리하고 그 역할을 준다는게 어떤 건지 잘 모르겟어요 제가 로직을 잘못 이해하고 있나요?

In GitLab by @ajeng518 on May 5, 2024, 01:47

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

아뇨 그 로직 맞습니다 메소드는 일단 수정할게요 저도 헷갈리네요. 그리고 4번의 경우 생각하시는게 맞습니다. 사실 크게 구분을 안해주어도 프론트에서도 문제는 사실 없습니다! 그러면 이 부분도 void 반환으로 보내고 그냥 성공 실패 여부만 알려주는 방식이 더 괜찮을 것 같다고 생각하시나용?

In GitLab by @tnatna0801 on May 5, 2024, 02:07

Commented on Backend/src/main/java/com/homegravity/Odi/domain/party/service/PartyService.java line 221

일단 지금 제 머리로는 그렇슴다 아정씨도 한번더 생각해보셨음 합니다 저는 아무래도 그냥 코드 잠깐 본 사람이라 틀릴 수 있어용

근데 애초에 방장만 가능한 api 아닙니까? 신청자와 참여자가 보이는 버튼이 다르다가 흠 어떤 건가요? 방장만 내보내기 할 수 있으니까 방장만 버튼이 보인다는 건가용? 그런거면 동승글 구인 상세 조회에서 이 api를 요청한 요청자의 role을 반환해주고 또 방장일 경우와 아닐 경우 신청자 목록 조회에 차이가 있게해두었습니당 아무튼 추가적인 제한 기능?은 아마 동승자 구인 글 상세 조회에 추가하시면 될 것 같네용 거기가 이 거절하기 기능 전에 요청되는 부분이니까용

In GitLab by @ajeng518 on May 5, 2024, 12:02

added 6 commits

b3949f64...0990f21d - 5 commits from branch develop
b0ac1b29 - Merge branch 'develop' into 'BE/feat/party-join-accept'

Compare with previous version

In GitLab by @ajeng518 on May 5, 2024, 12:22

mentioned in commit 17bef5a22857ab6e8c0289880c021233d34c13ec

Gravity-of-home / ODI