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

[feat] 알림 세팅 및 요청 관련 알림 구현 #152

Merged
merged 3 commits into from
Aug 26, 2023
Merged

[feat] 알림 세팅 및 요청 관련 알림 구현 #152

merged 3 commits into from
Aug 26, 2023

Conversation

psyeon1120
Copy link
Member

@psyeon1120 psyeon1120 commented Aug 24, 2023 •
edited
Loading

🌱 이슈 번호

close #139


💬 기타 사항

  • 요청하기, 요청 상태 변경하기에는 적용시켜놨고, 2달 이후 알림은 나중에 스케줄러 사용할 예정입니다.
  • 그리고 User 엔티티에 notification 컬럼이 필요한가요? 피그마 보면 알림 수신 여부 선택하는 건 없더라구여. 필요없는거면 지우겠습니당.
  • 로그아웃을 한 사용자라도 나중에 로그인했을 때 그동안 온 알림도 볼 수 있게 일단 알림 목록에 저장하게 했습니당(알림을 보내진 않아요)
  • 비밀 파일이 생겼으니 다들 추가해놓으세용
  • 프론트랑 연결을 다음시간에....

@psyeon1120 psyeon1120 added 🔮 feat 기능을 생성해요 ! ☀️ seoyeon 서연이가 만든 기능이에요 ! 🔔 notifications 알림 기능이에요 ! labels Aug 24, 2023
@psyeon1120 psyeon1120 self-assigned this Aug 24, 2023
chaerlo127
chaerlo127 approved these changes Aug 25, 2023
View reviewed changes
src/main/kotlin/com/psr/psr/order/entity/OrderStatus.kt
Comment on lines +8 to +11
ORDER_WAITING("요청대기", null),
PROGRESSING("진행중", "요청이 진행되었습니다"),
COMPLETED("진행완료", "요청이 진행 완료되었습니다"),
CANCELED("요청취소", "요청이 취소되었습니다");
Copy link
Collaborator

Choose a reason for hiding this comment

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

👏👏👏

Copy link
Member

Choose a reason for hiding this comment

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

🤍

@sojungpp sojungpp closed this Aug 26, 2023
@sojungpp sojungpp reopened this Aug 26, 2023
sojungpp
sojungpp approved these changes Aug 26, 2023
View reviewed changes
Copy link
Member

@sojungpp sojungpp left a comment

Choose a reason for hiding this comment

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

피그마 업뎃하면 언급드리겠습니당 수고했으요 ✨

@psyeon1120 psyeon1120 merged commit ca3fe55 into develop Aug 26, 2023
1 check passed
@psyeon1120 psyeon1120 deleted the feat/#139-notification branch August 26, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sojungpp sojungpp sojungpp approved these changes

@chaerlo127 chaerlo127 chaerlo127 approved these changes

Assignees

@psyeon1120 psyeon1120

Labels
🔮 feat 기능을 생성해요 ! 🔔 notifications 알림 기능이에요 ! ☀️ seoyeon 서연이가 만든 기능이에요 !
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feat] 알림 세팅 및 요청 관련 알림 구현
3 participants
@psyeon1120 @sojungpp @chaerlo127