✨ feat: application API 파일 생성 및 타입 정의 #87
Conversation
Yun-Jinwoo
left a comment
Yun-Jinwoo
left a comment
There was a problem hiding this comment.
status에 'pending' | 'accepted' | 'rejected' | 'canceled' 과 'accepted' | 'rejected' | 'canceled' 2가지 union type 이 있는데 현재는 그냥 객체 안에 적었습니다만 별개로 분리하는 게 좋을지 의견 구해봅니다
각 union type들이 한번만 사용되는 것 같으니 굳이 분리하지 않아도 될 것 같다는 생각입니다!
alert에서 사용하는 application은 형태가 조금 달라 applicationApi 파일에 넣지 않고 그대로 alertApi 파일에 남겼는데 분리해서 파일에 넣는게 좋을까요?
확인해본 결과 alertApi의 ApplicationItem과 ApplicationInfo는 현재 파일과 좀 다르네요!
이런 경우 분리하기엔 네이밍도 애매한 것 같아서 그대로 사용해도 괜찮다고 생각합니다!
| export interface ApplicationUserResponse { | ||
| offset: 'number'; | ||
| limit: 'number'; | ||
| count: 'number'; | ||
| hasNext: 'boolean'; | ||
| items: ApplicationUserInfo[]; | ||
| link: LinkInfo[]; | ||
| } | ||
|
|
||
| // GET /shops/{shop_id}/notices/{notice_id}/applications - 가게의 특정 공고의 지원 목록 조회 response | ||
| export interface ApplicationNoticeResponse { | ||
| offset: 'number'; | ||
| limit: 'number'; | ||
| count: 'number'; | ||
| hasNext: 'boolean'; | ||
| items: ApplicationNoticeInfo[]; | ||
| link: LinkInfo[]; | ||
| } |
There was a problem hiding this comment.
타입에 ''가 붙어있습니다! 수정 부탁드립니다!
There was a problem hiding this comment.
그냥 복붙하다 보니 놓쳤나 보네요.. 수정하였습니다!
일단 api 파일 내에서는 일단 중복적으로 사용되지는 않는데, 혹시나 Component처럼 외부 파일에서 쓸 수 있지 않을까~ 해서 묻게 되었습니다! |
📌 변경 사항 개요
application API type들을 정의했습니다.
📝 상세 내용
🔗 관련 이슈
Resolves: #68
🖼️ 스크린샷(선택사항)
💡 참고 사항
'pending' | 'accepted' | 'rejected' | 'canceled'과'accepted' | 'rejected' | 'canceled'2가지 union type 이 있는데 현재는 그냥 객체 안에 적었습니다만 별개로 분리하는 게 좋을지 의견 구해봅니다applicationApi파일에 넣지 않고 그대로alertApi파일에 남겼는데 분리해서 파일에 넣는게 좋을까요?