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

Feat [#194] 스프린트 QA alarm관련 수정사항 반영 #195

Merged
merged 5 commits into from
Jul 2, 2024

Conversation

binisnil
Copy link
Contributor

@binisnil binisnil commented Jul 1, 2024

👻 PULL REQUEST

💻 작업한 내용

  • Badge관련 작업 진행했습니다
  • 사용자가 알림을 읽음에 따라 badge값을 -1 시켜주는 작업, 노티창을 확인했을 때 모든 badge의 개수를 0으로 초기화해주는 작업을 위해, 서버 선생님들과 으쌰하여 API 추가하고, 연동 완료했습니다!
  • 기존 QA에서 전달받았던 알림센터에 푸시가 쌓이지 않는 현상은 제쪽에서 발견할 수 없어서,,, 결국 수정하지 못했습니다. 만약에 해당 현상 발견하시면, 어떤 상황에서 발생했는지 자세히 알려주시면 감사하겠습니다!

💡 참고사항

  • 현재 Package.resolved파일 수정사항(origin hash)이 발생했는데, 이 부분때문에 빌드 오류가 나면 말해주세요!

📸 스크린샷

기능 스크린샷
GIF

📟 관련 이슈

Copy link
Member

@boogios boogios left a comment

Choose a reason for hiding this comment

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

p3
알림 기능 구현하느라 고생많으셨습니다!!!! 완전 좋네여~~!!!

@@ -33,4 +35,22 @@ final class DontBePushAlarmHelper {
func checkUserLoginState() {

}

func patchFCMBadgeAPI(badge: Int) async throws -> BaseResponse<EmptyResponse>? {
Copy link
Member

Choose a reason for hiding this comment

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

p3
와... 새로운 API 까지 만들면서 구현하셨군요!!! 짱 수고하셨습니다!!! 멋지네영🔥

if let contentID = notiInfomation["relateContentId"] as? String,
let aps = notiInfomation["aps"] as? [String: Any],
let badge = aps["badge"] as? Int {
let pushAlarmHelper = DontBePushAlarmHelper(contentID: Int(contentID) ?? 0,
Copy link
Member

Choose a reason for hiding this comment

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

p3
짱짱입니다!!!!!

@binisnil binisnil merged commit 52c665b into develop Jul 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 4차 스프린트 QA 사항 반영 - 알림 관련 버그 수정
2 participants