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

[feat] 온보딩 ui 변경 #127

Merged
merged 25 commits into from
Mar 25, 2024
Merged

[feat] 온보딩 ui 변경 #127

merged 25 commits into from
Mar 25, 2024

Conversation

l2zh
Copy link
Contributor

@l2zh l2zh commented Mar 20, 2024

📌 관련 이슈

📝 Work Desciption

  • 진입 시점 플로우 변경 하고 상태 처리 해두었습니다
  • 디자인 변경사항 반영 하였습니다.

📚 Reference 혹은 궁금한 사항들

화이팅~~

@l2zh l2zh added feat 구현 및 개선 UI UI 작업 Pull Request 🔥 pr 날릴때 준희 👻 막내온탑 labels Mar 20, 2024
@l2zh l2zh added this to the 2차 스프린트 milestone Mar 20, 2024
@l2zh l2zh self-assigned this Mar 20, 2024
Copy link
Collaborator

@Jokwanhee Jokwanhee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~~~ 🔥

이슈!

  • Network 클래스에서 리스너가 사용되는 것 같아요! 그런데 해당 리스너를 제때 해제해주지 않아서 메모리 누수 나는 것 같은데 한 번 확인 부탁드립니다! (전역으로 사용되니 어쩔 수 없나 싶기도합니다..)
  • 요거는 궁금한 거에요!! -> inviteCode 를 StateFlow로 사용해서 Databinding 하던데, 여기서 한 번씩 빌드할 때, 에러납니다.. clean project하면 괜찮아지는데, flow-databinding의 호환성 문제일까요..?

@l2zh
Copy link
Contributor Author

l2zh commented Mar 20, 2024

고생하셨습니다~~~~ 🔥

이슈!

  • Network 클래스에서 리스너가 사용되는 것 같아요! 그런데 해당 리스너를 제때 해제해주지 않아서 메모리 누수 나는 것 같은데 한 번 확인 부탁드립니다! (전역으로 사용되니 어쩔 수 없나 싶기도합니다..)
  • 요거는 궁금한 거에요!! -> inviteCode 를 StateFlow로 사용해서 Databinding 하던데, 여기서 한 번씩 빌드할 때, 에러납니다.. clean project하면 괜찮아지는데, flow-databinding의 호환성 문제일까요..?
  1. 콜백해제를 까먹었네요! 감삼다~
  2. 이건 이유를 잘 모르겠네요ㅠㅠ stateFlow 지원된게 꽤 전인데 왜 이럴까요ㅋㅋㅋ

@l2zh l2zh merged commit 908b8bd into develop Mar 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 구현 및 개선 Pull Request 🔥 pr 날릴때 UI UI 작업 준희 👻 막내온탑
Projects
Development

Successfully merging this pull request may close these issues.

[ui] 온보딩 컬러 변경
2 participants