-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FEAT] api 이쁘게 #6
Conversation
Visit the preview URL for this PR (updated for commit cdb4ece): https://batshu-c203f--pr6-feature-api-9qipgjrq.web.app (expires Thu, 28 Sep 2023 01:59:50 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 83b3acd7145a633c2e47144814099f2df4c43e54 |
src/domain/apiPaths.ts
Outdated
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.
이거 경로 별도로 따는 이유는 msw입니당
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.
이거 queryWrapper 쓸거면 템플릿타입으로 아예 타입도 정하는게 좋지 않을까.
default 타입으로 record쓰고
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.
원래 따로 뒀다가 response타입도 다 여기있길래 여기에 넣음 따로 둘거면 다 옮기는 쪽으로 안그럼 똑같은 타입 두번 정의해야됨..
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.
@HYEOK9 어디 공통으로 dto만 정의하고 import하는 방향은 어떤가요!
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.
일단 endpoint만 빼놓음 옮기고 싶은거 있음 옮기고 머지 ㄱ
No description provided.