[마이그레이션] RTK Query 마이그레이션 #109
Merged
Conversation
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
…api-migration [마이그레이션] tanstack query 기초 세팅 및 club api 변경
[마이그레이션] club, category, forceUpdate, benefit 관련 api 마이그레이션
…ang-s-h [어드민]history,member,notice,owner,ownerRequest,review,user 마이그레이션
…gration-3-kang-s-h Revert "[어드민]history,member,notice,owner,ownerRequest,review,user 마이그레이션"
[마이그레이션] RTK Query -> Tanstack Query 마이그레이션 - abtest, address, auth, banner, bannerCategory
Tanstack Query 마이그레이션 - room, shop, shopMenu, updateList, upload
[오류 수정] 불필요한 import 삭제
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
ff1451
approved these changes
Oct 2, 2025
Contributor
ff1451
left a comment
ff1451
left a comment
There was a problem hiding this comment.
새롭게 구조 적용하는거라 힘들었을텐데 참여하신 분들 모두 고생하셨습니다!
혜준님, 동훈님, 성훈님 작업은 개별 pr에서 확인했어서 도훈님 부분만 보고 궁금한 점이 있어 질문 하나 남겼습니다!
Comment on lines
+30
to
+40
| const { mutate: changePasswordMutation } = useMutation({ | ||
| mutationFn: changePassword, | ||
| onSuccess: () => { | ||
| message.success('비밀번호 변경 완료'); | ||
| handleModalClose(); | ||
| }, | ||
| onError: (error) => { | ||
| message.error(error.message || '비밀번호 변경 실패'); | ||
| }, | ||
| }); | ||
|
|
Contributor
There was a problem hiding this comment.
다른 mutation 쿼리들은 별도 파일로 분리하여 관리하는 것을 보이는데 비밀번호 변경 하나뿐이어서 별도로 관리하지 않는 것이 맞는지 질문드립니다
Contributor
Author
There was a problem hiding this comment.
맞습니다! mutation은 자유롭게 적용하도록 했습니다.
해당 훅을 재사용한다면, 별도로 분리하여 사용하고 그렇지 않다면 컴포넌트에 직접 정의해서 사용하도록 했습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What is this PR? 🔍
Changes 📝
폴더 구조
마이그레이션 담당
✔️ Please check if the PR fulfills these requirements
developbranch unconditionally?main?yarn lint