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

Fix/#216 라운드 대진 횟수 설정 에러 해결 #218

Merged
merged 4 commits into from
Nov 18, 2023
Merged

Conversation

navyjeongs
Copy link
Member

🤠 개요

💫 설명

  • 대진표 수정 버튼을 누르면 에러 뜨던 문제가 있었는데 서버의 response type이 변경되어서 발생한 문제여서 서버의 응답 타입에 맞게 타입을 변경해서 해결했어요.
  • 8강의 횟수를 3회에서 5회로 수정하면 32강에 적용되던 문제가 있었어요. 그 이유가 서버에서 8강 16강 32강 데이터를 주면 프론트에서 32강 16강 8강으로 변경해서 데이터를 보여주고 횟수를 수정하면 다시 8강 16강 32강으로 변경해서 서버에 요청을 해야하는데 중간중간 reverse 메서드를 사용했는데 reverse 메서드는 원본 배열을 수정해서 발생하던 문제였어요.
  • 따라서, 깊은 복사를 통해서 원본 배열을 건드리지 않고 reverse를 사용했어요.

📷 스크린샷 (Optional)

@navyjeongs navyjeongs added the 🛠 Fix 프로젝트 버그 수정 이슈 label Nov 5, 2023
@navyjeongs navyjeongs requested a review from pp449 November 5, 2023 14:31
@navyjeongs navyjeongs self-assigned this Nov 5, 2023
@navyjeongs navyjeongs linked an issue Nov 5, 2023 that may be closed by this pull request
1 task
@navyjeongs navyjeongs changed the title Fix/#216 Fix/#216 라운드 대진 횟수 설정 에러 해결 Nov 5, 2023
Copy link
Member

@pp449 pp449 left a comment

Choose a reason for hiding this comment

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

굿굿 ~~

Comment on lines 24 to 26
cacheTime: 0,
staleTime: 0,
});
Copy link
Member

Choose a reason for hiding this comment

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

cacheTime도 0으로 설정한 이유가 있나요??

Copy link
Member Author

Choose a reason for hiding this comment

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

무의식적으로 같이 추가했는데 삭제할게요!

@navyjeongs navyjeongs merged commit 099f4cb into dev Nov 18, 2023
1 check passed
@navyjeongs navyjeongs deleted the Fix/#216 branch November 18, 2023 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Fix 프로젝트 버그 수정 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix: 라운드 대진횟수 설정시 에러 해결
2 participants