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.

Sign up for GitHub

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

Jump to bottom

[Feature/Notifications] 알림 기능 구현 #66

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
Open

[Feature/Notifications] 알림 기능 구현 #66

wants to merge 9 commits into from

Conversation

kyubumjang
Copy link
Member

@kyubumjang kyubumjang commented Jun 23, 2022 •
edited
Loading

📌 과제 설명

알림 기능 구현했습니다!

👩‍💻 요구 사항과 구현 내용

댓글 작성 및 좋아요를 누르면 해당 게시물의 작성자에게 알림이 생성되어 전송됩니다!

  • 사용자가 좋아요 누르기

스크린샷 2022-06-23 오후 5 42 39

  • 포스트의 작성자에게 알림 생성, 새로고침 시 확인 가능

스크린샷 2022-06-23 오후 5 42 42

  • 알림 클릭 시 해당 포스트로 이동

스크린샷 2022-06-23 오후 5 43 01

  • Post 확인 시 알림 seen 처리

스크린샷 2022-06-23 오후 5 43 07

  • 댓글 작성 시 알림 생성

스크린샷 2022-06-23 오후 5 51 42

  • 포스트 주인 알림 확인 가능

스크린샷 2022-06-23 오후 5 51 37

@kyubumjang
Squashed commit of the following:
4397a6f 
commit caeed41
Merge: f472f86 987e72f
Author: Lee Jaeung <[email protected]>
Date:   Wed Jun 22 16:43:40 2022 +0900

    Merge pull request #63 from prgrms-fe-devcourse/refactor/user-profile

    [Refactor/UserProfile] 유저 프로필 게시물 스켈레톤 처리 추가

commit f472f86
Merge: 6b0d938 caf0eaa
Author: Lee Jaeung <[email protected]>
Date:   Wed Jun 22 16:34:53 2022 +0900

    Merge pull request #62 from prgrms-fe-devcourse/refactor/All

    [Style/All] 페이지 전반에 걸친 디자인 수정 및 폰트 적용

commit 987e72f
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 16:28:22 2022 +0900

    feat(UserProfile): 유저 프로필 아이템 스켈레톤 처리 추가

commit caf0eaa
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 16:10:07 2022 +0900

    refactor(EditProfile, Post): 게시물 업로드, 이미지 변경 시 alert 추가

commit 4a1e803
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:56:15 2022 +0900

    refactor(alert): alert 텍스트 일관되게 유지

commit 6b0d938
Author: Paeng <[email protected]>
Date:   Wed Jun 22 15:49:20 2022 +0900

    docs: README 수정

    2차 수정

commit cb1aa31
Author: Paeng <[email protected]>
Date:   Wed Jun 22 15:43:08 2022 +0900

    docs: README 수정

    1차 정리

commit a655780
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:33:42 2022 +0900

    design(NavigationBar): 폰트 수정을 위한 HTML 구조 변경 및 정렬 변경

commit 2d4d76e
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:26:15 2022 +0900

    design(LoginForm): 패딩 수정

commit 3193d9a
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:24:22 2022 +0900

    design(FormField): 폰트 수정

commit 0e61a16
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:19:54 2022 +0900

    design(Form): 이미지 크기 수정

commit ba36050
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:18:22 2022 +0900

    design(App): 같은 폰트를 사용하는 태그 병합

commit a2aac74
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:14:24 2022 +0900

    design(All): 인라인으로 적용된 폰트 스타일 삭제 및 전체 적용
@kyubumjang
feat(NotificationsPage): 알림 페이지 UI 구성
95b335b
@kyubumjang
feat(Notifications): 알림 API 연동
f899b81
@kyubumjang
feat(Notifications): 라우팅 추가
0ed06c2
@kyubumjang
feat(Badge): Badge 추가
d4c4574
@kyubumjang
feat(Skeleton): Notification 스켈레톤 추가
2a9d123
@kyubumjang
feat(Notificaiton): API 연동 및 기능 구현
7c4bc57
@kyubumjang
Squashed commit of the following:
f08525b 
commit cede9f8
Author: qq8721443 <[email protected]>
Date:   Wed Jun 22 21:29:25 2022 +0900

    refactor(Controversy): 투표 시 replace 설정

commit 82a892d
Author: qq8721443 <[email protected]>
Date:   Wed Jun 22 18:02:15 2022 +0900

    refactor(Logout): Logout 방법 변경

commit 12a8684
Author: qq8721443 <[email protected]>
Date:   Wed Jun 22 17:39:45 2022 +0900

    refactor(Loading): Loader 위치 수정 및 프로필 사진 변경 버그 수정

commit 394bb5d
Merge: caeed41 7755ebe
Author: Jeongki <[email protected]>
Date:   Wed Jun 22 17:18:20 2022 +0900

    Merge pull request #64 from prgrms-fe-devcourse/refactor/result-page-loading-state

    [Refactor] ResultPage 댓글 로딩, 페이지 로딩 상태 추가

commit 7755ebe
Merge: 8ce72ba caeed41
Author: Jeongki <[email protected]>
Date:   Wed Jun 22 17:10:08 2022 +0900

    Merge branch 'develop' into refactor/result-page-loading-state

commit 8ce72ba
Author: qq8721443 <[email protected]>
Date:   Wed Jun 22 17:05:36 2022 +0900

    chore(ResultPage): develop 최신 변경사항 반영

commit cca5039
Author: qq8721443 <[email protected]>
Date:   Wed Jun 22 16:51:06 2022 +0900

    refactor(ResultPage): 댓글 로딩, 페이지 로딩 상태 추가

commit e264919
Author: qq8721443 <[email protected]>
Date:   Wed Jun 22 16:50:39 2022 +0900

    refactor(Skeleton): Skeleton 컴포넌트 스타일 적용 변경

commit caeed41
Merge: f472f86 987e72f
Author: Lee Jaeung <[email protected]>
Date:   Wed Jun 22 16:43:40 2022 +0900

    Merge pull request #63 from prgrms-fe-devcourse/refactor/user-profile

    [Refactor/UserProfile] 유저 프로필 게시물 스켈레톤 처리 추가

commit f472f86
Merge: 6b0d938 caf0eaa
Author: Lee Jaeung <[email protected]>
Date:   Wed Jun 22 16:34:53 2022 +0900

    Merge pull request #62 from prgrms-fe-devcourse/refactor/All

    [Style/All] 페이지 전반에 걸친 디자인 수정 및 폰트 적용

commit 987e72f
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 16:28:22 2022 +0900

    feat(UserProfile): 유저 프로필 아이템 스켈레톤 처리 추가

commit caf0eaa
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 16:10:07 2022 +0900

    refactor(EditProfile, Post): 게시물 업로드, 이미지 변경 시 alert 추가

commit d2d4f9d
Author: qq8721443 <[email protected]>
Date:   Wed Jun 22 16:01:32 2022 +0900

    chore(README.md): README 수정

commit 4a1e803
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:56:15 2022 +0900

    refactor(alert): alert 텍스트 일관되게 유지

commit 6b0d938
Author: Paeng <[email protected]>
Date:   Wed Jun 22 15:49:20 2022 +0900

    docs: README 수정

    2차 수정

commit cb1aa31
Author: Paeng <[email protected]>
Date:   Wed Jun 22 15:43:08 2022 +0900

    docs: README 수정

    1차 정리

commit a655780
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:33:42 2022 +0900

    design(NavigationBar): 폰트 수정을 위한 HTML 구조 변경 및 정렬 변경

commit 2d4d76e
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:26:15 2022 +0900

    design(LoginForm): 패딩 수정

commit 3193d9a
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:24:22 2022 +0900

    design(FormField): 폰트 수정

commit 0e61a16
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:19:54 2022 +0900

    design(Form): 이미지 크기 수정

commit ba36050
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:18:22 2022 +0900

    design(App): 같은 폰트를 사용하는 태그 병합

commit a2aac74
Author: DESKTOP-H8C5V9L\ekzm5 <[email protected]>
Date:   Wed Jun 22 15:14:24 2022 +0900

    design(All): 인라인으로 적용된 폰트 스타일 삭제 및 전체 적용
qq8721443
qq8721443 reviewed Jun 25, 2022
View reviewed changes
Copy link
Member

@qq8721443 qq8721443 left a comment

Choose a reason for hiding this comment

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

규범님 알림 기능까지 구현하시다니... 정말 고생 많으셨습니다! 리뷰가 조금 늦었네요 😭
NavigationBar 에서 알림 버튼에 뱃지 출력하는 부분에 궁금한 점이 있어 코멘트 남겼습니다! 확인해 주시면 정말 감사드리겠습니다~! 👍👍👍👍

toronto/src/components/organisms/NavigationBar/index.js
color: 'inherit',
}}
>
{user.notifications.length === 0 ? (
Copy link
Member

Choose a reason for hiding this comment

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

이 부분에서 단순히 사용자가 받은 알림의 길이에 따라 뱃지를 보여줄지 말지를 결정하고 있는 것 같은데, 사용자 입장에서 받은 알림이 있지만 알림을 읽었을 때에도 뱃지가 떠 있는 건 UX 측면에서는 좋지 않은 것 같습니다! 해당 부분을 알림 배열의 seen 이 false 인 값이 있을 때에만 뱃지를 출력하는 건 어떨까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qq8721443 qq8721443 qq8721443 left review comments

@GeonWooPaeng GeonWooPaeng Awaiting requested review from GeonWooPaeng

@jaeung-E jaeung-E Awaiting requested review from jaeung-E

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kyubumjang @qq8721443