Feat/#499 알림메시지 변경하기 #511

BackFoxx · 2023-10-19T04:37:09Z

🔥 연관 이슈

close: 알림 메시지를 설정할 수 있는 API 만들기 #499

📝 작업 요약

원하는 알림 항목의 알림 메시지를 수정할 수 있는 API를 제공한다.

🔎 작업 상세 설명

PUT 메서드로 /admin/notification/message 를 호출하여 알림 메시지를 수정할 수 있습니다.
권한 설정이 되어있지 않으므로 uri만 알고 있다면 누구나 호출할 수 있습니다.
알림 메시지에 %title, %comment, %count를 포함하는 경우 각각 포스트 제목, 댓글 내용, 따봉 개수로 치환해 표시합니다.
예) [%title] 글에 댓글이 달렸습니다. -> [아 소고기 먹고싶다~~~] 글에 댓글이 달렸습니다.

🌟 리뷰 요구 사항

예상치 못한 버그가 있을 수 있읍니다.

# Conflicts: # backend/src/main/java/edonymyeon/backend/member/application/MemberService.java # backend/src/main/java/edonymyeon/backend/member/ui/MemberController.java # backend/src/test/java/edonymyeon/backend/member/docs/MemberControllerDocsTest.java

# Conflicts: # backend/src/main/java/edonymyeon/backend/auth/application/AuthService.java

…_회원_정보_수정 # Conflicts: # backend/src/main/java/edonymyeon/backend/member/ui/MemberController.java # backend/src/test/java/edonymyeon/backend/auth/application/AuthServiceTest.java # backend/src/test/java/edonymyeon/backend/member/docs/MemberControllerDocsTest.java

…에 대한 식별자로 사용

# Conflicts: # backend/src/docs/asciidoc/common/mypage-api.adoc # backend/src/main/java/edonymyeon/backend/image/domain/Domain.java # backend/src/main/java/edonymyeon/backend/member/application/MemberService.java # backend/src/main/java/edonymyeon/backend/member/domain/Member.java # backend/src/main/java/edonymyeon/backend/member/ui/MemberController.java # backend/src/main/java/edonymyeon/backend/post/application/PostReadService.java # backend/src/test/java/edonymyeon/backend/member/application/MemberServiceTest.java # backend/src/test/java/edonymyeon/backend/member/docs/MemberControllerDocsTest.java # backend/src/test/java/edonymyeon/backend/member/domain/MemberTest.java # backend/src/test/java/edonymyeon/backend/member/integration/MemberIntegrationTest.java

github-actions · 2023-10-19T04:40:37Z

Unit Test Results

  69 files +  2   69 suites +2 34s ⏱️ +4s
367 tests +10 361 ✔️ +4 0 💤 ±0 6 ❌ +6
370 runs +10 364 ✔️ +4 0 💤 ±0 6 ❌ +6

For more details on these failures, see this check.

Results for commit 9748a1f. ± Comparison against base commit d176d38.

♻️ This comment has been updated with latest results.

jyeost

안녕하세여 여우!!!!
잘 해주셨군여...
제가 시켜놓고 리뷰 너무 늦게봐서 죗성합니다....

pr이 알림 메세지를 추가 & 어드민으로 알림 메세지를 변경
이 두가지 기능을 담고 있으니 읽기 힘들더라고여...
나눠서 줬으면 조앗을거 같은데,,,,, ㅜㅜ

그리고... 제목 길이나 댓글 길이가 길면 ... 으로 축약해달라고 부탁드렸는데
어디에 구현 되어 있는지 못찾았어여...ㅜㅜ 알려주심 감사홥돠

backend/src/main/resources/db/migration/mysql/V1.4.0__notification_content.sql

jyeost · 2023-10-30T08:19:19Z

backend/src/main/java/edonymyeon/backend/notification/ui/NotificationController.java

+
+    @ApiVersion(from = "1.0")
+    @PutMapping("/admin/notification/content")
+    public ResponseEntity<Void> updateNotificationMessage(@RequestBody NotificationContentRequest notificationContent) {


나중에 어드민이 커지면 그쪽 컨트롤러로 옮겨가는 게 성격이 맞아보며유

맞아요 어드민 권한도 만들어야 하고.. 회원 권한을 관리하는 방법은 회의를 통해 정해야겠는데욥

jyeost · 2023-10-30T08:20:09Z

backend/src/main/java/edonymyeon/backend/notification/domain/Notification.java

 import lombok.NoArgsConstructor;

 @NoArgsConstructor
-@Getter
 @Entity


getter를 직접 만들어 준 이유가 있나요??

조만간 롬복 파기작업을 할거에요 .. 사용하지 않는 게터가 도메인 객체에 너무 많은 건 좋지 못한 것 같소

backend/src/main/java/edonymyeon/backend/notification/application/NotificationService.java

backend/src/main/java/edonymyeon/backend/member/repository/MemberRepository.java

backend/src/main/java/edonymyeon/backend/notification/application/NotificationService.java

.../edonymyeon/backend/notification/domain/notification_content/domain/NotificationContent.java

hectick

🦊👏
좀만 더 고생해주십시옹!!

hectick · 2023-10-30T10:38:48Z

backend/src/main/java/edonymyeon/backend/notification/ui/NotificationController.java

+    @ApiVersion(from = "1.0")
+    @PutMapping("/admin/notification/content")
+    public ResponseEntity<Void> updateNotificationMessage(@RequestBody NotificationContentRequest notificationContent) {


admin용에는 api 버전을 어떻게 붙여줘야할까요??
무조건 버전 1부터 시작되도록?
api 헤더가 없어도 무방하긴 한데..

아예 어드민을 다른 모듈로 빼버리는건 어떤가여?

요건 혼자서 잘 못하겠어요 🥹 화요일에 같이 해주라

backend/src/main/resources/db/migration/mysql/V1.4.0__notification_content.sql

...kend/notification/domain/notification_content/application/NotificationMessageRepository.java

backend/src/docs/asciidoc/common/notification-api.adoc

...edonymyeon/backend/notification/application/NotificationService_NotificationMessageTest.java

backend/src/main/java/edonymyeon/backend/notification/application/NotificationService.java

This2sho

고생이 많네요 여우상
이리내랑 케로가 열심히 남겨줘서 궁금한 부분만 커맨트 남겼습니다!
저는 어프로브하고 푸시하면 확인할게염👍

backend/src/main/java/edonymyeon/backend/notification/domain/Notification.java

.../edonymyeon/backend/notification/domain/notification_content/domain/NotificationContent.java

This2sho · 2023-10-31T01:42:37Z

backend/src/main/java/edonymyeon/backend/notification/ui/NotificationController.java

+    @ApiVersion(from = "1.0")
+    @PutMapping("/admin/notification/content")
+    public ResponseEntity<Void> updateNotificationMessage(@RequestBody NotificationContentRequest notificationContent) {


아예 어드민을 다른 모듈로 빼버리는건 어떤가여?

backend/src/main/java/edonymyeon/backend/notification/domain/Notification.java

# Conflicts: # backend/src/test/resources/truncate.sql

This2sho and others added 30 commits September 13, 2023 05:30

refactor: 프로필 이미지도 정팩메로 통일

e5494fd

refactor: imageFileUploader 메소드 분리

aec11af

feat: 회원 정보 수정 기능 구현

1666e78

test: api 문서화 테스트 코드 작성

5b8153e

test: 최신 RequestBuilder로 변경

817324a

refactor: ImageInfo 생성과 실제 이미지 저장 로직 분리

7c4abb8

refactor: 중복 체크 AuthService 에서 MemberService로 이동

3f6af80

fix: conflict 해결, 변경된 API 인해 깨지는 테스트 해결

b33321e

fix: application.properties 프로필 수정

a00f599

test: 프로필 이미지 테스트용 픽스쳐 추가

98f7c69

refactor: api 수정에 따른 회원 정보 수정 기능 변경

0c0a7cd

docs: 회원 정보 수정 docs 테스트 구현

8070bfa

Merge remote-tracking branch 'origin/dev' into feat/#53_회원_정보_수정

16722da

# Conflicts: # backend/src/main/java/edonymyeon/backend/auth/application/AuthService.java

refactor: Domain 객체에서 url 반환하도록 변경

a84aa0c

refactor: MyPage에서 profileImage 반환할 수 있게 변경 및 response 필드명(id) 통일

6500efc

feat: 회원 프로필 이미지 반환하도록 변경 및 docs 수정

b686e12

fix: dto 필드명 바꾸면서 바뀐 것들 다시 수정

7e54a52

refactor: 마이페이지 조회 v1, v2 대응하도록 수정

bb527ed

refactor: 마이페이지 DTO 변경으로 인한 v2도 대응하도록 변경

2fa9348

test: 이미지 파일 생성시 Post용, Profile용 분리

f633889

test: Docs 테스트에 대응하는 버전 적용

619267d

refactor: 잘못 적용된 예외 수정

74636f6

build: 사용하지 않는 의존성 제거

7eb8fc9

refactor: 알림 발송시 사용하는 제목과 본문 정보를 묶어 하나의 엔티티로 분리

b46242e

feat: 알림 메시지로 발송할 제목과 본문 내용을 수정하는 기능 구현

e202771

refactor: NotificationMessage의 이름을 NotificationMessageId로 바꾸고, 알림 메시지…

81674b4

…에 대한 식별자로 사용

refactor: 알림 메시지가 변경된 후에 보내는 알림에만 변경된 알림 메시지가 적용되도록 개선

f4b6b77

BackFoxx added 10 commits October 18, 2023 14:53

Merge branch 'dev' into feat/#499_알림메시지_변경하기

2153e97

feat: 알림 메시지를 수정하는 API 구현

5f681c0

refactor: NotificationMessageId의 이름과 패키지 위치 수정

b4a51cf

fix: 테스트 깨짐 방지를 위해 기본 알림 메시지를 저장

214c444

feat: 네임드 파라미터를 사용해 title과 body 값을 동적으로 치환할 수 있는 기능 구현

ca76e03

refactor: 컨트롤러의 파라미터로 DTO를 받도록 변경

74dede0

Merge branch 'dev' into feat/#499_알림메시지_변경하기

79e5a3c

docs: notification 도메인 API 문서화

4e741a9

docs: notification 도메인 flyway 적용

f9d932a

BackFoxx added ❤️ Backend for BackEnd ⚡️ feature New feature or request labels Oct 19, 2023

BackFoxx added this to the 6차 데모데이 milestone Oct 19, 2023

BackFoxx requested review from This2sho, hectick and jyeost October 19, 2023 04:37

BackFoxx self-assigned this Oct 19, 2023

jyeost requested changes Oct 30, 2023

View reviewed changes

hectick requested changes Oct 30, 2023

View reviewed changes

This2sho approved these changes Oct 31, 2023

View reviewed changes

Merge remote-tracking branch 'origin/dev' into feat/#499_알림메시지_변경하기

9748a1f

# Conflicts: # backend/src/test/resources/truncate.sql

BackFoxx closed this Nov 3, 2023

BackFoxx deleted the feat/#499_알림메시지_변경하기 branch November 3, 2023 13:49

BackFoxx restored the feat/#499_알림메시지_변경하기 branch November 4, 2023 04:42

BackFoxx reopened this Nov 4, 2023

BackFoxx mentioned this pull request Nov 5, 2023

Feat/#499 알림메시지 변경하기 2 #531

Open

BackFoxx closed this Nov 5, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/#499 알림메시지 변경하기 #511

Feat/#499 알림메시지 변경하기 #511

BackFoxx commented Oct 19, 2023

github-actions bot commented Oct 19, 2023 •

edited

Loading

jyeost left a comment

jyeost Oct 30, 2023

BackFoxx Nov 3, 2023

jyeost Oct 30, 2023

BackFoxx Nov 3, 2023

hectick left a comment

hectick Oct 30, 2023

This2sho Oct 31, 2023

BackFoxx Nov 5, 2023

This2sho left a comment

This2sho Oct 31, 2023

Feat/#499 알림메시지 변경하기 #511

Feat/#499 알림메시지 변경하기 #511

Conversation

BackFoxx commented Oct 19, 2023

🔥 연관 이슈

📝 작업 요약

🔎 작업 상세 설명

🌟 리뷰 요구 사항

github-actions bot commented Oct 19, 2023 • edited Loading

Unit Test Results

jyeost 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

hectick 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

This2sho left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Oct 19, 2023 •

edited

Loading