Fix/184/dashboard, invitations, members querykey 개선#185
Merged
Conversation
1 task
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ToKyun02
approved these changes
Feb 24, 2025
Owner
ToKyun02
left a comment
ToKyun02
left a comment
There was a problem hiding this comment.
쿼리 키 개선하신거 확인했습니다!
오늘 팀미팅에서 예기한 것 처럼 객체로 넘겨서 관리하는 방식이 안헷갈리고 더 범용적으로 쓸 수 있는 것 같아서 저도 참고 많이했습니다!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❓이슈
✍️ Description
리액트쿼리 쿼리키 관련 공부를 더하고 쿼리키 사용방식을 개선했습니다.
기존에 받은 params object를 그대로 넘겨서 사용할때,
invalidateQuries에서 쿼리키로 무효화가 잘 안되는 현상이 있었습니다.
params object를 그대로 useQuery의 키로 사용한게 문제가 된것이 아니라,
invalidateQuries에서 쿼리키를 작성하는 방식이 잘못되었던것을 알게 되어서, 다시 한번 개선 pr을 올립니다.
(invalidateQuries에서도 똑같은 형태로 객체로 넣어야 했습니다. (객체로 부분값만 넣어도 partial로 무효화가 가능하네요))
✅ Checklist
PR
Test
Additional Notes