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] [#229] 웨이팅 탭 api들 연결 #230

Merged
merged 16 commits into from
Aug 24, 2024

Conversation

wjdtkdgns777
Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 commented Aug 23, 2024

화면 캡처 2024-08-23 230118

웨이팅탭의 api 연결 및 마이너한 버그들을 수정하였습니다

아마 큰 문제는 없을듯 한데

  1. 웨이팅 리스트가 없을때 부스 및 주점 보러가기 버튼은 어차피 맵으로 가는거다보니 간단하게 popBackStack을 재활용했습니다
  2. 웨이팅 취소할때 각 아이템에서 취소버튼을 누르면 uistate의 waitingCancelDialogWaitingId를 변경해주고, 그걸 웨이팅 dialog에서 취소 확인을 누를때 이용해서 취소하도록 했는데 flow가 자연스러운거같긴 한데 한번 봐주셨음 합니다. 로직상 waitingCancelDialogWaitingId를 매번 사용후 다시 초기화 안해도 문제 생기진 않을건데 전반적으로 혹시 모르니..
  3. 또 cancelBoothWaiting이거 PUT으로 취소하는거라 결과값으로 오는걸 사용하거나 하진 않아서 간략히 처리했는데.. 이것도 문제될건 없을거같네요
  4. 웨이팅탭에 pull to refresh가 있어야 할거같아 디자인은 없는데 구현은 했습니다. 근데 못생겼습니다.. ㅋㅋㅋㅋㅋㅋ 커스텀으로 하려했는데 왜인지 안뜨고(??) 디자인도 없어서 일단 기본기능으로 했는데 안예쁘네요. 흠.. 이부분은 머터리얼2에 있다해서 해당부분 추가해서 사용했습니다.

그리고 renovate 봇으로 올린 ktlint오류나서 다시 낮췄습니다.
요정도만 봐주시면 될듯 합니다

@wjdtkdgns777 wjdtkdgns777 self-assigned this Aug 23, 2024
@wjdtkdgns777 wjdtkdgns777 added the enhancement New feature or request label Aug 23, 2024
@wjdtkdgns777 wjdtkdgns777 linked an issue Aug 23, 2024 that may be closed by this pull request
@easyhooon
Copy link
Collaborator

easyhooon commented Aug 23, 2024

웨이팅탭에 pull to refresh가 있어야 할거같아 디자인은 없는데 구현은 했습니다. 근데 못생겼습니다.. ㅋㅋㅋㅋㅋㅋ 커스텀으로 하려했는데 왜인지 안뜨고(??) 디자인도 없어서 일단 기본기능으로 했는데 안예쁘네요. 흠.. 이부분은 머터리얼2에 있다해서 해당부분 추가해서 사용했습니다.

material3 에도 해당 기능을 지원합니다. 전에 구현한 경험이 있어서요
https://developer.android.com/reference/kotlin/androidx/compose/material3/pulltorefresh/package-summary

https://github.com/easyhooon/FavoriteFeed/blob/master/feature/home/src/main/kotlin/com/kenshi/favoritefeed/feature/home/HomeScreen.kt
아마 이거로 구현하면 조금은 더 이쁘긴할거라서 migration 해주시면 좋을 것 같습니다.

@easyhooon
Copy link
Collaborator

다른 부분들은 크게 문제될 부분은 없을 것 같네요

@wjdtkdgns777
Copy link
Collaborator Author

어쩐지 말이 안되긴 했어요 ㅋㅋㅋ Compose pull to refresh 검색해서 나온 최신글들이 다 material2 pullRefresh길래 의아했는데 역시군요. https://google.github.io/accompanist/swiperefresh/ 여기서도 deprecated 되었으니 https://developer.android.com/reference/kotlin/androidx/compose/material/pullrefresh/package-summary 요거 쓰라길래 당연히 요거 쓰면 되겠거니 했는데 Compose material3 pull to refresh 검색하니 나오는군요 이런~

@wjdtkdgns777
Copy link
Collaborator Author

사소한 오류들좀 수정했습니다

@easyhooon
Copy link
Collaborator

굿 입니다.

@wjdtkdgns777 wjdtkdgns777 merged commit 5517e35 into develop Aug 24, 2024
1 check passed
@wjdtkdgns777 wjdtkdgns777 deleted the feature/waiting-tab-connection branch August 24, 2024 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

웨이팅 탭 api들 연결
2 participants