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

[AN/USER] fix: 사용자 불편사항 및 오동작 해결(#980 #983 #984) #981

Merged
merged 5 commits into from
May 19, 2024

Conversation

SeongHoonC
Copy link
Member

@SeongHoonC SeongHoonC commented May 19, 2024

📌 관련 이슈

✨ PR 세부 내용

바텀 네비게이션 정책 수정

바텀네비게이션으로 화면 이동 시 백스택 초기화합니다.
기존 검색 크래시 발생 오류도 해결됩니다.

축제 목록 새로고침

다른 화면에 이동했다가 돌아오면 생명주기 콜백 함수가 재실행 되면서 뷰모델 함수가 재호출됩니다.
success 이고 refresh 가 아니면 재요청하지 않도록 변경했습니다.

아티스트, 학교 상세 이전 축제 보기

처음 시작이 진행중, 진행 예정 목록이면 과거 축제를 요청하지 않습니다.
첫 요청 결과에 따라

  1. last 인지
  2. 리스트 아이템이 없는지
  3. 마지막이 진행중, 진행예정 아이템인지

확인하고 추가 요청합니다.

@SeongHoonC SeongHoonC added AN 안드로이드에 관련된 작업 USER labels May 19, 2024
@SeongHoonC SeongHoonC self-assigned this May 19, 2024
@github-actions github-actions bot requested review from EmilyCh0 and re4rk May 19, 2024 05:31
Copy link
Member

@EmilyCh0 EmilyCh0 left a comment

Choose a reason for hiding this comment

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

LGTM!

@SeongHoonC SeongHoonC changed the title [AN/USER] 바텀 네비게이션 클릭 시 쌓였던 BackStack 이 초기화된다. (#980) [AN/USER] 바텀 네비게이션 클릭 시 쌓였던 BackStack 이 초기화된다. (#980 #983 #984) May 19, 2024
Copy link
Collaborator

@re4rk re4rk left a comment

Choose a reason for hiding this comment

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

생각보다 복잡한 문제였군요 ㅋㅋ
LGTM!

@SeongHoonC SeongHoonC changed the title [AN/USER] 바텀 네비게이션 클릭 시 쌓였던 BackStack 이 초기화된다. (#980 #983 #984) [AN/USER] 사용자 불편사항 및 오동작 해결(#980 #983 #984) May 19, 2024
@SeongHoonC SeongHoonC changed the title [AN/USER] 사용자 불편사항 및 오동작 해결(#980 #983 #984) [AN/USER] fix: 사용자 불편사항 및 오동작 해결(#980 #983 #984) May 19, 2024
@github-actions github-actions bot requested review from EmilyCh0 and re4rk May 19, 2024 11:31
@SeongHoonC SeongHoonC merged commit efc9ebc into dev May 19, 2024
3 checks passed
@SeongHoonC SeongHoonC deleted the feat/#980 branch May 19, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AN 안드로이드에 관련된 작업 USER
Projects
Status: Done
3 participants