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.

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

refactor : 알림 스펙 변경사항 최산화 #683

Merged

Conversation

chws0508
Copy link
Collaborator

@chws0508 chws0508 commented Oct 5, 2023

#️⃣연관된 이슈

#682

📝작업 내용

알림 스펙사항 변경 최산화 헀습니다.

@chws0508 chws0508 added the Android 안드로이드 관련 이슈 label Oct 5, 2023
@chws0508 chws0508 added this to the 6차 스프린트 milestone Oct 5, 2023
@chws0508 chws0508 self-assigned this Oct 5, 2023
@chws0508 chws0508 merged commit 364112c into android-main Oct 5, 2023
1 check passed
chws0508 added a commit that referenced this pull request Oct 7, 2023
* refactor : 알림 스펙 변경사항 최산화

* refactor : 알림 스펙 변경사항 최신화(#683)

* refactor : 대댓글 알림 목록 조회 API 수정

* refactor : LocalDateTime 의 RelativeTime 몇분전, 몇시간전 표시 로직 해결

* refactor : Notification 읽음 여부 업데이트 API 매서드를 Put 에서 Patch 로 변경

* refactor : 알림보관함 화면 댓글 알림 RelativeDateTime Pattern 을 사용하도록 변경

* refactor : 알림 보관함에서 최신 순으로 알림을 정렬

* refactor : LocalDateTimeExt 수정

* refactor : 불필요한 주석 제거

* refactor : UpdatedNotification -> Notification 으로 변경, NotificationType 을 Enum으로 관리

* refactor : notificationInformation -> additionalInformation 으로 변경

* refactor : requireNotNul 로 오류를 체크하도록 변경

* refactor : ktlint 충돌 해결 및 additionalInformation -> extraNotificationInformation 으로 수정
@chws0508 chws0508 deleted the HotFix/#682-알림_스펙_사항_변경_적용 branch October 11, 2023 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android 안드로이드 관련 이슈
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant