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

Feature/fix qa final #52

Merged
merged 4 commits into from
Aug 23, 2024
Merged

Feature/fix qa final #52

merged 4 commits into from
Aug 23, 2024

Conversation

bywindow
Copy link
Collaborator

2024.08.22 QA 수정

  • 보드 화면 하단 BottomView의 요일 정렬 추가
  • 프로필 생성 완료 다이얼로그 한번만 뜨도록 수정
  • 프로필 생성 완료 다이얼로그 캐릭터 그림자 없도록 하고 사이즈 수정
  • 프로필 생성/수정 화면 캐릭터 아이템들 사이즈 수정

Copy link
Member

@eshc123 eshc123 left a comment

Choose a reason for hiding this comment

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

프로필 생성 완료 다이얼로그 한번만 뜨게 성공했구나!
이번 경우도 그렇고, 보드 화면에서 캐릭터 애니메이션도 그렇고
이벤트 컨트롤이랑 화면 전환이랑 엮일 때를 주의해서 구현을 해야겠어..!
고생 많았어!

@bywindow
Copy link
Collaborator Author

프로필 생성 완료 다이얼로그 한번만 뜨게 성공했구나! 이번 경우도 그렇고, 보드 화면에서 캐릭터 애니메이션도 그렇고 이벤트 컨트롤이랑 화면 전환이랑 엮일 때를 주의해서 구현을 해야겠어..! 고생 많았어!

이거 사실 내가 바보였던 게 dialog 뜨는 이벤트라서 SharedFlow로 했어야 했는데 StateFlow로 해가지고 상태가 캐싱되어 있었어...ㅋㅋㅋ

@bywindow bywindow merged commit ce866e4 into dev Aug 23, 2024
1 check passed
@bywindow bywindow deleted the feature/fix-qa-final branch August 23, 2024 02:12
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