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

#682 알림 스펙 사항 변경 #689

Merged
merged 14 commits into from
Oct 7, 2023

Conversation

chws0508
Copy link
Collaborator

@chws0508 chws0508 commented Oct 5, 2023

#️⃣연관된 이슈

ex) #682

📝작업 내용

알림 스펙사항 변경에 대해 대처하였습니다

  • 행사 알림에서 이제 제목이 보이도록 수정하였습니다.
  • 모든 알림함 목록에 대해서 같은 형식으로 받고, notificationInfromation 항목은 먼저 String으로 받은 후에따로 파싱을 해주어야 합니다.

예상 소요 시간 및 실제 소요 시간

1시간/ 2시간

… HotFix/#682-알림_스펙_사항_변경_적용

# Conflicts:
#	android/2023-emmsale/app/src/main/java/com/emmsale/presentation/service/KerdyFirebaseMessagingService.kt
@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
Copy link
Member

@tmdgh1592 tmdgh1592 left a comment

Choose a reason for hiding this comment

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

알림 스펙 사항 변경하시느라 고생하셨습니다 스캇!
예비군 이슈로 함께 작업에 참여하지 못한 부분이 있기 때문에 코멘트를 비교적 많이 달아놓았습니다.

확인해주시고 코멘트 부탁드립니다 : ) 👍🏻

@chws0508 chws0508 requested a review from tmdgh1592 October 7, 2023 07:36
Copy link
Member

@tmdgh1592 tmdgh1592 left a comment

Choose a reason for hiding this comment

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

@chws0508 리뷰 반영하시느라 고생하셨습니다 스캇!
간단한 코멘트 하나를 남겼으니 확인해주시고 반영여부는 스캇이 결정해주세요 : )
approve 하겠습니다.

+) CI가 통과하지 못하고 있습니다. 확인 후 수정 부탁드립니다 👍

@chws0508 chws0508 merged commit 7e51450 into android-main Oct 7, 2023
1 check passed
@chws0508 chws0508 deleted the HotFix/#682-알림_스펙_사항_변경 branch October 11, 2023 06:30
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.

2 participants