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] #426 - 네비게이션 오류 해결 #428

Merged
merged 2 commits into from
Mar 14, 2023

Conversation

JIWON1923
Copy link
Collaborator

관련 이슈

문제가 일어난 이유와 해결 방법

  • iOS 16.1부터 추천 모음집 작성 시 앱이 꺼졌던 이유는 Navigation Stack Path 모델을 전달해주지 않았기 때문이었습니다.
  • Navigation Path 초기화의 경우 iOS 16.1부터 작동하도록 수정했습니다.
  • iOS 16.0애서 추천 모음집 작성 시 빈 화면이 나왔던 이유는 Navigation Router에 해당 Data 정보가 없었기 때문입니다.
  • 현재 추천모음집이 Full Screen Cover로 presented라는 바인딩 변수가 필요합니다. 하지만 저희 프로젝트의 경우 Navigation Router를 통해 데이터를 한 번에 관리하고 있어서 함수를 오버로딩 하여 문제를 해결했습니다.
  • 이와 같이 데이터가 누락되는 문제를 해결하기 위해서는 Navigation Data 전체를 다루는 열거형으로 정의하는 것이 좋을 것 같습니다

구현/변경 사항

  • 추천 모음집 작성시 앱이 종료되는 문제 (iOS 16.1부터)해결
  • 추천 모음집 작성시 빈 페이지가 나오는 문제 (iOS 16) 해결

스크린샷, iOS 16.0 (Navigation View)

2023-03-14.4.59.13.mov

스크린샷2, iOS 16.1 * (Navigation Stack)

16.2.mov

@JIWON1923 JIWON1923 added Type-BugFix Fix a bug or error 🚨 Emergency Need fast work and approve 🍔 내일은개발천재 labels Mar 14, 2023
@JIWON1923 JIWON1923 added this to the 9th Sprint - HotFix milestone Mar 14, 2023
@JIWON1923 JIWON1923 self-assigned this Mar 14, 2023
Copy link
Member

@HanGyeongjun HanGyeongjun left a comment

Choose a reason for hiding this comment

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

빠른 문제해결! 수고하셨습니다
궁금한 점은 단축어 작성 과정에서는 왜 이 에러가 발생하지 않았던 걸까요..? environmentobject는 있었지만 같은 fullscreencover로 나타나서 같은 오류가 발생할 것 같은데🤔

@JIWON1923
Copy link
Collaborator Author

빠른 문제해결! 수고하셨습니다
궁금한 점은 단축어 작성 과정에서는 왜 이 에러가 발생하지 않았던 걸까요..? environmentobject는 있었지만 같은 fullscreencover로 나타나서 같은 오류가 발생할 것 같은데🤔

단축어 작성 과정에 네비게이션이 사용되지 않기 때문입니다!
지금은 네비게이션 코드 없이 Full screen 자체로 해결되고 있어서 문제가 발생하지 않았네요
만약 단축어가 예전처럼 여러 단계로 작성되어야했다면, 같은 문제가 발생했을거예요!
레거시 코드가 참 많네요..! 이슈 끊어놓을게요~!

@jim4020key
Copy link
Collaborator

테스트 코드의 필요성 😭..?? 빠른 해결 👏 수고하셨습니다!

@JIWON1923 JIWON1923 merged commit 2ecd61f into develop Mar 14, 2023
@JIWON1923 JIWON1923 deleted the bugfix/426-curationWritingError branch March 14, 2023 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 Emergency Need fast work and approve Type-BugFix Fix a bug or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 추천 모읍집 작성 불가
4 participants