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

안내면진다 목록 기획, 디자인 변경사항 반영 및 버그 픽스 #688

Open
wants to merge 7 commits into
base: develop-frontend
Choose a base branch
from

Conversation

cys4585
Copy link
Contributor

@cys4585 cys4585 commented Oct 18, 2024

PR의 목적이 무엇인가요?

안내면진다 목록 기획, 디자인 변경사항 반영 및 버그 픽스

이슈 ID는 무엇인가요?

설명

  • 서비스 이름 변경: 안내면진다 -> 룰렛
  • 목록 UI 변경
  • 데이터 페칭 폴링으로 변경
    • 폴링으로 주기적으로 페칭을 해도, 데이터가 변한 게 없어서 리렌더링이 발생하지 않는 문제 -> useQuery가 isFetching을 반환하도록 하는 것 만으로 리렌더링이 트리거된다. 그러나 리렌더링이 두 번 발생하는 낭비가 있었어요(사소하긴 해). useQuery 내부에서 leftMinute을 계산해서 bets 데이터에 포함해서 넣어주도록 해서, leftMinute 값이 변경되었을 때만 리렌더링이 트리거되도록 했습니다. (소파 짱)
  • 룰렛 생성 validation
  • 따닥 문제(중복 생성 문제) 해결
  • 새로고침 문제 해결
  • 페이지 이탈, 접근 문제 해결

@cys4585 cys4585 added FE 프론트엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) 🚑 버그 fix (develop에서 파생된 문제) labels Oct 18, 2024
@cys4585 cys4585 self-assigned this Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) 🚑 버그 fix (develop에서 파생된 문제)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant