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

[GGFE-55] 게임 결과 목록 버그 수정 #801

Merged

Conversation

yoouyeon
Copy link
Member

📌 개요

  • 게임 결과 목록 테스트 중 발견된 버그 "일부" 수정했습니다.

💻 작업사항

  • 전체 게임 목록에서 시즌 드롭다운이 보이는 문제
    → radioMode 조건이 잘못되어있어 수정했습니다.
  • 랭크 게임 목록에서 api 요청이 안되는 문제
    → query key 값을 잘못 전달하고 있어서 수정했습니다.
  • 목록 조회 시에 더보기를 눌렀을 때 Error: Invalid src prop 발생
    → next.config.js domain에 'cdn.intra.42.fr' 추가해주었습니다.

❗️ 전체 / 랭크 게임 목록에서 유저 검색 시에도 전체 게임이 나오는 문제는 api response 자체에 문제가 있어서 확인 부탁드린 상태입니다...

✅ 변경로직

@yoouyeon yoouyeon self-assigned this May 31, 2023
Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

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

변경사항 확인했습니다!

Copy link
Contributor

@42sungwook 42sungwook left a comment

Choose a reason for hiding this comment

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

확인했습니다. 테스트해봤는데 잘되네요👍

@42sungwook 42sungwook merged commit aebb4b0 into GGFE-00/Feat/api-refactoring Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants