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: 상세 화면 축제 목록 요청 로직 변경 및 Throwable 예외 처리 제거(#986) #987

Merged
merged 9 commits into from
May 20, 2024

Conversation

SeongHoonC
Copy link
Member

📌 관련 이슈

✨ PR 세부 내용

상세 화면 축제 목록 요청 로직 변경

서버로 isPast == true 를 요청할 때
lastFestivalId, lastLocalDate 가 past 가 아니면 데이터가 제대로 반환되지 않습니다.
그래서 그 로직을 수정했습니다.

추가로 10 개 요청을 20개 요청으로 늘렸습니다. (서버는 페이징이 되고 있지 않기 때문에..)

30개로 늘리면 400 에러가 발생합니다 참고해주세요!

Throwable 을 던지지 않도록 코드 제거

@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 13:01
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 merged commit 549c7f3 into dev May 20, 2024
1 check passed
@SeongHoonC SeongHoonC deleted the feat/#986 branch May 20, 2024 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AN 안드로이드에 관련된 작업 USER 🐛 버그 버그에 관련된 작업
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[AN] 아티스트 상세에서 같은 축제가 두 번 보이는 문제 해결
3 participants