Skip to content

[Feat] #73 - Push 알림 탭 시 작성하기로 화면전환#86

Merged
L-j-h-c merged 7 commits intoTeamRecorDream:developfrom
L-j-h-c:feature/#73-pushTapTransition
Dec 30, 2022
Merged

[Feat] #73 - Push 알림 탭 시 작성하기로 화면전환#86
L-j-h-c merged 7 commits intoTeamRecorDream:developfrom
L-j-h-c:feature/#73-pushTapTransition

Conversation

@L-j-h-c
Copy link
Copy Markdown
Contributor

@L-j-h-c L-j-h-c commented Dec 30, 2022

👻 작업한 내용

Push 알림 탭 시 작성하기로 화면전환 기능을 구현했습니다.
Push 알림을 탭하면 작성하기 화면으로 자동 전환됩니다.

🎤 PR Point

해당 기능을 구현하며 가장 크게 고려한 점은, application state와 인증 플로우입니다.
푸시 알림을 탭한 경우라고 무작정 작성하기로 보내면 인증되지 않은 사용자가 들어올 수 있기 때문에, 기존에 splashVC에 존재하는 인증 플로우를 이용하고자 했습니다.

1. Active 상태에서는 재귀함수를 이용해서 MainTabBar를 찾고, dreamWriteVC가 present되도록 했습니다.

2. InActive 상태에서도 마찬가지로 이미 인증이 되었을 것이라 가정하고, 1과 같은 방식으로 해결했습니다.

3. 아예 앱이 꺼진 상태에서 들어오는 경우, sceneDeleagate에서 userDefault 값을 바꾸어 주었습니다.

4. 앱의 특정 뷰를 찾아가는 기능을 구현하니 확실히 코디네이터의 필요성이 느껴졌슴다... 나중에 리팩토링 해보면 좋겠네요!

📹 영상

Active

KakaoTalk_Video_2022-12-31-03-48-39.mp4

앱이 꺼진 상태

KakaoTalk_Video_2022-12-31-03-48-45.mp4

📮 관련 이슈

@L-j-h-c L-j-h-c added the feat 구현·개선 사항에 관련된 내용입니다. label Dec 30, 2022
@L-j-h-c L-j-h-c self-assigned this Dec 30, 2022
Copy link
Copy Markdown
Member

@Suyeon9911 Suyeon9911 left a comment

Choose a reason for hiding this comment

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

Flow 깔끔하게 잘 하셨네용 !! 정말 이 부분에선 코디네이터가 역할을 잘 해줬을 것 같네요 !! 혹시 그러면 코디네이터 쓰게 되면 굳이 재귀함수로 찾지 않아도 되는건가요??

아 그리고 shouldShowWrite의 userDefalut 값이 false면 그냥 Splash 인증 플로우로 이동되는건가용 ??

@L-j-h-c
Copy link
Copy Markdown
Contributor Author

L-j-h-c commented Dec 30, 2022

@Suyeon9911
저도 이런 구현은 많이 해보지 않아서 정확히는 알고 있지는 않지만, coordinator가 있으면 각 뷰컨트롤러간의 관계가 정리되고, deep link(이동해야 할 타겟이 여러 개인 경우 열거형으로 정의)에 대한 분기처리를 통해서 각 coordinator들의 행동을 결정해 놓을 수 있기 때문에 좋은 것 같습니다... 여기를 참조하셔도 될 것 같고,, 저도 더 공부해봐야 할 것 같아요!

지금 제가 재귀함수로 구현한 방식은 가능한 모든 경우의 수를 계산해서 재귀로 구현했는데, coordinator가 있으면 각 deep link의 상황에서 각자의 coordinator가 해야할 행동만 지정해주면 되는 그런 느낌이라고 생각합니다...!

@L-j-h-c L-j-h-c merged commit 777b0cb into TeamRecorDream:develop Dec 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 구현·개선 사항에 관련된 내용입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] 푸시 알림 탭 시 작성하기 뷰로 화면 전환시키기

2 participants