콕 찌르기 및 알림 전송 추가 #69

JJ503 · 2024-02-17T16:37:16Z

closed [Feat] 콕 찌르기 및 알림 전송 추가 #68

콕 찌르기 기능의 경우 알림을 보내는 게 끝이기에 다른 이슈로 빼기가 애매하다 판단해 함께 진행하겠습니다.
콕 찌르기 및 알림 관련 내용에 대해선 3차 스프린트 1주차 회의록에 작성해 두었습니다. 확인해 주시면 감사하겠습니다!
또한, TODO를 통해 고민을 한 가지 작성했는데 내용은 아래와 같습니다.
코드는 TODO를 검색해 확인하시면 보다 편하게 확인하실 수 있습니다.

추가로, 팀원인지 확인하는 로직에 대해 Goal에서 확인할 수 있도록 했습니다.
해당 부분이 어떤지 확인하신 후 의견 주시면 감사하겠습니다.
또한, 괜찮다면 효선님께서도 해당 메서드를 통해 검증하셔도 좋을 것 같습니다!

고민

콕 찌르기 API의 컨트롤러 코드가 URI 상 골 패키지 내에 있는 것이 적절하다고 생각했는데, 어떻게 생각하시는지 궁금합니다.
팀원 목록을 현재는 NotificationService에서 가져오도록 하고 있는데, 이를 Goal을 통해 수행하는 것이 더 적절할까요?

jhsseonn

고생 많으셨습니다!
대부분 간단한 컨벤션 수정이고, 골 초대 알림 보내기의 경우 골 팀원 수정시에 추가된 팀원에게도 보내야해서 추가로 구현이 필요할 것 같아 rc로 남깁니다.
그리고 아래는 질문주신 고민에 대한 답변입니다.

콕 찌르기 uri는 저도 좋은 것 같습니다. 코멘트에도 적어두었지만, 지금의 uri가 프론트쪽에서도 구별이 쉬울 것 같습니다. 오히려 스탬프 조회쪽에서도 이렇게 uri를 설정하는 것이 좋은지 고민하게 되네요..! 스탬프도 goals/{goalId}/stamp 와 같은 형식이 나을지 질문드립니다!
팀원 목록을 가져오는 로직이 Goal과 관련된 로직이어서 저는 Goal에 있으면 좋을 것 같습니다!

jhsseonn · 2024-02-19T10:53:13Z

src/main/java/com/backend/blooming/goal/application/PokeService.java

+    public void poke(final PokeDto pokeDto) {
+        final User sender = getUser(pokeDto.senderId());
+        final User receiver = getUser(pokeDto.receiverId());
+        final Goal goal = getGaol(pokeDto.goalId());


오타가 있네요!

jhsseonn · 2024-02-19T10:58:34Z

src/main/java/com/backend/blooming/goal/presentation/PokeController.java

+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
+
+


개행이 하나 더 들어갔네요!

jhsseonn · 2024-02-19T11:03:10Z

src/main/java/com/backend/blooming/goal/presentation/PokeController.java

+
+    private final PokeService pokeService;
+
+    // TODO: 2/17/24 API의 URI 상 이 패키지가 적절하다고 생각했는데, 괜찮을까요?


전 좋은 것 같습니다! 확실히 잘 구별되는 것 같아서 좋은 것 같습니다.
다만 제가 이번 스탬프 조회 기능을 구현하면서 stamps/{goalId} 이런 식으로 api 패키지를 설정했는데, 부적절하다고 판단되면 수정하도록 하겠습니다!

해당 리뷰를 먼저 보지 못하고, 효선님 pr을 리뷰해 동일한 의견이 있는 줄 몰랐네요..!
저도 효선님과 동일하게, /goals/{goalId}/stamps가 더 자연스럽지 않나? 하는 의문이 있었습니다.
그래서 uri 상 위 형태가 되면 더 좋겠다는 생각을 했습니다.
다만, 그랬을 때 스탬프 관련 코드들을 골로 옮겨야 할까? 하는 고민이 있었습니다.
해당 부분에 대해 어떻게 생각하시나요?

p.s. 스탬프와 관련된 이야기니, 해당 pr에 의견을 작성해주시면, 나중에 다시 확인하고 싶을 때 빨리 찾을 수 있을 것 같습니다!

jhsseonn · 2024-02-19T11:19:02Z

src/main/java/com/backend/blooming/goal/application/GoalService.java


    public Long createGoal(final CreateGoalDto createGoalDto) {
        final List<User> users = getUsers(createGoalDto.teamUserIds());
        final Goal goal = persistGoal(createGoalDto, users);

+        notificationService.sendRequestGoalNotification(goal);


골 팀 목록 수정시에도 sendRequestGoalNotification을 해줘야할 것 같습니다!
이 부분은 골 수정/삭제 부분 머지가 늦어지는 바람에 develop에 포함이 안되어 있었던 것 같네요...

그렇네요!
그런데, 해당 부분을 진행하려면 update()를 통해 추가된 팀원이 누구인지 알아야 합니다.
그래서 updateTeams()를 void가 아닌 List<GoalTeam>을 반환하게 하면 어떨지 의견 구하고 싶습니다.
해당 부분은 효선님의 의견을 구한 뒤 결정된 방법으로 수정 진행하려 합니다.
또한, 다른 update 메서드들과는 다르게 updateTeams만 반환값이 있어도 어떻게 생각하시는 지 궁금합니다.

좋습니다! �수정 로직 구현 과정에서 추가된 팀원을 반환해야하는 경우가 딱히 없을거라 판단해 반환값을 주지 않았는데 이 경우에는 수정된 내용을 반환받아야겠네요. updateTeams와 updateTeams()에 반환값을 부여하는 것에 동의합니다. 그럼 이 부분은 제가 리팩토링 pr을 파서 수정을 하겠습니다!

간단한 수정 사항이기에 이번 pr에서 제가 진행하도록 하겠습니다!

jhsseonn · 2024-02-19T11:26:41Z

src/test/java/com/backend/blooming/goal/application/PokeServiceTestFixture.java

+        final User 콕_찌르기_요청자 = User.builder()
+                                   .oAuthId("12345")
+                                   .oAuthType(OAuthType.KAKAO)
+                                   .name(new Name("사용자1"))
+                                   .email(new Email("[email protected]"))
+                                   .build();
+        final User 콕_찌르기_수신자 = User.builder()
+                                   .oAuthId("12346")
+                                   .oAuthType(OAuthType.KAKAO)
+                                   .name(new Name("사용자2"))
+                                   .email(new Email("[email protected]"))
+                                   .build();
+        final User 팀원이_아닌_사용자 = User.builder()
+                                    .oAuthId("12347")
+                                    .oAuthType(OAuthType.KAKAO)
+                                    .name(new Name("사용자3"))
+                                    .email(new Email("[email protected]"))
+                                    .build();


final 붙여줘야겠군요...! 리팩토링 진행하면서 수정하도록 하겠습니다!

jhsseonn · 2024-02-19T11:28:58Z

src/test/java/com/backend/blooming/notification/application/NotificationServiceTest.java

 import com.backend.blooming.user.infrastructure.repository.UserRepository;
-import org.assertj.core.api.SoftAssertions;
+import org.assertj.core.api.*;


별이 보이네요!

# Conflicts: # src/main/java/com/backend/blooming/exception/ExceptionMessage.java # src/main/java/com/backend/blooming/exception/GlobalExceptionHandler.java # src/main/resources/static/docs/goal.html

jhsseonn

수정사항 모두 확인했습니다! approve 하겠습니다!

JJ503 · 2024-02-22T22:18:33Z

위 코멘트에서 이야기한 대로, 골 업데이트에 대해 반환값을 만들어, 이에 해당하는 사용자들에게만 알림을 보내도록 수정했습니다.

sonarqubecloud · 2024-02-22T22:20:09Z

Quality Gate passed

Issues
28 New issues

Measures
0 Security Hotspots
97.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

JJ503 added 4 commits February 17, 2024 22:28

feat: 콕 찌르기 서비스 기능 추가 및 콕 찌르기 알림 전송 기능 추가

3d61c83

feat: 콕 찌르기 컨트롤러 기능 추가

2934ca4

feat: 친구 수락 시 알림 전송 로직 추가

a72cd0e

feat: 골 초대 시 알림 전송 로직 추가

061c130

JJ503 added the feature 기능 추가 label Feb 17, 2024

JJ503 requested a review from jhsseonn February 17, 2024 16:37

JJ503 self-assigned this Feb 17, 2024

docs: 콕 찌르기 api 명세서 추가

d7d97bd

jhsseonn requested changes Feb 19, 2024

View reviewed changes

JJ503 added 4 commits February 20, 2024 15:52

style: 오탈자 수정

4b19e5a

refactor: todo 제거

b86cea3

test: import 문의 와일드 카드 제거

8c71663

ci: 브랜치 최신화

3644d0c

# Conflicts: # src/main/java/com/backend/blooming/exception/ExceptionMessage.java # src/main/java/com/backend/blooming/exception/GlobalExceptionHandler.java # src/main/resources/static/docs/goal.html

jhsseonn approved these changes Feb 21, 2024

View reviewed changes

JJ503 added 2 commits February 23, 2024 07:07

feat: 골 팀원 업데이트 시, 추가된 팀원에게 알림을 전송 로직 추가 및 이를 위한 골 알림 메서드 수정

78d7c4f

fix: 테스트가 실패하는 문제 해결

a6ba731

JJ503 merged commit ef623d7 into develop Feb 22, 2024
2 checks passed

JJ503 deleted the feature/68 branch February 22, 2024 22:21

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

콕 찌르기 및 알림 전송 추가 #69

콕 찌르기 및 알림 전송 추가 #69

JJ503 commented Feb 17, 2024 •

edited

Loading

jhsseonn left a comment

jhsseonn Feb 19, 2024

jhsseonn Feb 19, 2024

jhsseonn Feb 19, 2024

JJ503 Feb 20, 2024

jhsseonn Feb 19, 2024

JJ503 Feb 20, 2024 •

edited

Loading

jhsseonn Feb 21, 2024

JJ503 Feb 21, 2024

jhsseonn Feb 19, 2024

jhsseonn Feb 19, 2024

jhsseonn left a comment •

edited

Loading

JJ503 commented Feb 22, 2024

sonarqubecloud bot commented Feb 22, 2024

		import org.springframework.web.bind.annotation.RequestMapping;
		import org.springframework.web.bind.annotation.RestController;


		private final PokeService pokeService;

		// TODO: 2/17/24 API의 URI 상 이 패키지가 적절하다고 생각했는데, 괜찮을까요?

콕 찌르기 및 알림 전송 추가 #69

콕 찌르기 및 알림 전송 추가 #69

Conversation

JJ503 commented Feb 17, 2024 • edited Loading

고민

jhsseonn left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JJ503 Feb 20, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jhsseonn left a comment • edited Loading

Choose a reason for hiding this comment

JJ503 commented Feb 22, 2024

sonarqubecloud bot commented Feb 22, 2024

Quality Gate passed

JJ503 commented Feb 17, 2024 •

edited

Loading

JJ503 Feb 20, 2024 •

edited

Loading

jhsseonn left a comment •

edited

Loading