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

[Fix] 푸시알림 UserDefaults에 FeedID를 저장하는 로직에서 Combine 사용으로 수정 #362

Open
1 task
NEULiee opened this issue Apr 14, 2023 · 2 comments
Assignees
Labels
type: bug Something isn't working 늘이

Comments

@NEULiee
Copy link
Member

NEULiee commented Apr 14, 2023

할 일

  • 백그라운드 상태에서 푸시알림 탭했을 때 상세화면으로 이동이 안되는 현상 수정

구독 이후에도 구독 이전 value를 가져올 수 있는 CurrentValueSubject 특징으로
코디네이터가 생성되기 전이나 후에 상관없이 피드 상세 화면을 띄울 수 있도록 수정할 예정

테스트중!

123.mp4

image

@NEULiee NEULiee added type: bug Something isn't working 늘이 labels Apr 14, 2023
@NEULiee NEULiee self-assigned this Apr 14, 2023
@NEULiee
Copy link
Member Author

NEULiee commented May 4, 2023

아직 진행중..ㅠㅠ

@NEULiee
Copy link
Member Author

NEULiee commented May 14, 2023

기존에 백그라운드에서 푸시 탭했을 때 이동하지 않았던 에러 이유

  • 생각 : 탭바 구성 → moveDetailView

  • 실제 : 탭바 구성(async), moveDetailView

  • async로 configureTabBar 구현 하신 이유

    • 메인스레드에서 실행되어야 한다는 에러
    • Firebase의 currentUser가 non-atomic으로 thread-safe 하지 않다.
image

async로 구성되어야 하는 이유는 맞는데,, 이러면 notificationCenter나 Combine을 써도 의미가 없다.
HomeCoordinator에서 받기엔 다른 화면에서도 이동이 가능해야하고 ㅇㅅㅇ!!

sync로 동작하게 하면 가능은 하다만 문제가 있다.

  1. 메인스레드에서 동작되어야 한다. -> 이건 메인으로만 보내면 될 수도?
  2. 동영상처럼 메인 헤더가 안뜬다........? 단순히 header text label 인데..
default.mp4

또 다시 돌아오겠습니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working 늘이
Projects
None yet
Development

No branches or pull requests

1 participant