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

스플래시 화면 디자인에 맞게 , 애니메이션 자연스럽게 변경 #662

Merged
merged 5 commits into from
Oct 15, 2023

Conversation

EvergreenTree97
Copy link
Collaborator

@EvergreenTree97 EvergreenTree97 commented Oct 10, 2023

Overview (Required)

  • out animation에 hold를 적용하여, 이전 액티비티가 남아있도록 변경하고 자연스럽게 변경하였습니다.
  • @limsaehyun dynamicExamId가 null이 아닐 때에만, Activity Clear Flag를 add하도록 변경해야 이전 액티비티가 clear되지 않아서 자연스러운 애니메이션이 표현됩니다. 사이드이펙트 2220db4

Screenshot

default.webm

@limsaehyun
Copy link
Member

@EvergreenTree97
작업 감사합니다.
코멘트 남겨주신 dynamicLinkExamId의 경우 null이 아닐때만 보내도 될 것 같습니다. 공유하기 방금 돌려봤는데 작동 확인했습니다.
하지만 현재 앱 내에서 로그아웃 시도 시 온보딩 페이지가 아닌, 바로 메인페이지로 이동되는데 이 부분 개선되어야 할 것 같습니다!

@limsaehyun
Copy link
Member

CI failed된 것도 체크 부탁드려요!

@EvergreenTree97
Copy link
Collaborator Author

@EvergreenTree97 작업 감사합니다. 코멘트 남겨주신 dynamicLinkExamId의 경우 null이 아닐때만 보내도 될 것 같습니다. 공유하기 방금 돌려봤는데 작동 확인했습니다. 하지만 현재 앱 내에서 로그아웃 시도 시 온보딩 페이지가 아닌, 바로 메인페이지로 이동되는데 이 부분 개선되어야 할 것 같습니다!

로그아웃 문제는 제 PR부터 발생한 문제일까요? 로그아웃 관련 로직은 건든게 없어서, 제가 모르는 사이드 이펙이 혹시 존재하는지..?

@EvergreenTree97 EvergreenTree97 merged commit 566a82f into qa/1.3.1 Oct 15, 2023
1 of 2 checks passed
@EvergreenTree97 EvergreenTree97 deleted the fix/splash branch October 15, 2023 12:54
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.

2 participants