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] 매칭시 발생되는 로딩 UI 구현 및 적용 #70

Merged
merged 5 commits into from
Feb 15, 2024

Conversation

ham2174
Copy link
Collaborator

@ham2174 ham2174 commented Feb 15, 2024

관련 이슈

작업한 내용

  • 매칭시 발생되는 로딩 UI를 제작하였습니다.
  • 제작된 로딩 UI를 적용하였습니다.

PR 포인트

  • 로딩 UI를 :match에서 구현하였습니다.

    • 이유 : 로딩 UI가 matching 기능을 사용할때에만 특화된 UI로 :designsystem에서 구현하지 않고 :match에서 구현하였습니다. 추후 공통된 loading 화면이 추가적으로 발생되면 현재 구현된 로딩UI와 추후 제작된 로딩UI를 :designsystem으로 옮길지 논의가 필요합니다.
  • 기존에 MatchScreen()에서 UiState 분기처리를 하였던것을 발견했었는데, 로딩상태일때에도 상단 TopBar가 다른 모습으로 존재해서 MatchRoute()로 상태 분기를 옮겼습니다.
    image

🚀Next Feature

@ham2174 ham2174 added the ham😸 Good for newcomers label Feb 15, 2024
@ham2174 ham2174 requested a review from murjune February 15, 2024 11:49
@ham2174 ham2174 self-assigned this Feb 15, 2024
Copy link
Member

@murjune murjune left a comment

Choose a reason for hiding this comment

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

@ham2174 굿입니다!! 나중에 1차 릴리즈하고 나서 논의해봅시당!!
그리고 이거 바로 머지할게유~
제가 지금 작업하는 부분이랑 충돌이 날 것 같아서 지금 미리 합치려구요~

@murjune murjune merged commit 2e7f2d6 into develop Feb 15, 2024
1 check passed
@murjune murjune deleted the feature/create-loading-ui branch February 15, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ham😸 Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 로딩 UI 제작
2 participants