-
Notifications
You must be signed in to change notification settings - Fork 1
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
refactor : query key 수정 #877
The head ref may contain hidden characters: "feature/libraryApi_\uCFFC\uB9AC\uD0A4_\uD3EC\uB9F7_\uD1B5\uC77C_#866"
Conversation
src/api/libraryManageApi.ts
Outdated
book: () => [...libraryManageKeys.base, 'books'] as const, | ||
borrowInfo: () => [...libraryManageKeys.base, 'borrow-infos'] as const, | ||
bookManageList: (params: BookListSearch) => [...libraryManageKeys.book(), params] as const, | ||
bookDetail: (params: number) => [...libraryManageKeys.book(), params] as const, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
params 대신 bookId가 좋아보입니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
반영했습니다!
src/api/libraryManageApi.ts
Outdated
borrowLogList: (param: BorrowLogListSearch) => ['libraryManage', 'borrowLogList', param] as const, | ||
base: ['manage'] as const, | ||
book: () => [...libraryManageKeys.base, 'books'] as const, | ||
borrowInfo: () => [...libraryManageKeys.base, 'borrow-infos'] as const, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아마 카멜케이스로 하기로 했던 것 같습니다!
borrow-infos
-> borrowInfos
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
반영했습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굿!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
연관 이슈
작업 요약
�Library Manage Api 쿼리키 포멧 통일
작업 상세 설명
리뷰 요구사항
3분