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

HotFix: 앱 충돌 수정 #63

Merged
merged 1 commit into from
Jul 30, 2024
Merged

HotFix: 앱 충돌 수정 #63

merged 1 commit into from
Jul 30, 2024

Conversation

ShapeKim98
Copy link
Contributor

#️⃣연관된 이슈

#62

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 테스트 플라이트로 어플을 받았을 시에, 로그인에서 메인으로 넘어가는 상황에서 앱 충돌이 일어나는 것을 해결하고자 하였습니다.

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 로그인 화면을 구현할 당시 FullScreenCover로 사용될 것을 염두하고 dismiss코드를 넣었던게 원인이지 않았을까 하여 해당 코드를 제거하였습니다.
  • 시뮬레이터, 로컬 빌드 설치에는 충돌이 안일어나서 이번 pr을 머지하고 결과를 확인해볼 예정입니다.

close #62

@ShapeKim98 ShapeKim98 added the Bug 🔫 현재 발견된 버그를 수정하기 위함 label Jul 30, 2024
@ShapeKim98 ShapeKim98 requested review from ITlearning and stealmh July 30, 2024 09:33
@ShapeKim98 ShapeKim98 self-assigned this Jul 30, 2024
Copy link
Member

@stealmh stealmh left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~

@ShapeKim98 ShapeKim98 merged commit 1c78017 into develop Jul 30, 2024
1 check passed
@stealmh stealmh deleted the hotfix/#62-login-crash branch September 3, 2024 12:27
@stealmh stealmh linked an issue Oct 2, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🔫 현재 발견된 버그를 수정하기 위함
Projects
None yet
Development

Successfully merging this pull request may close these issues.

앱 충돌 이슈
2 participants