-
Notifications
You must be signed in to change notification settings - Fork 0
Docs : 동아리 즐겨찾기 추가 및 즐겨찾기 삭제 API 설계 문서 작성 #3
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
base: main
Are you sure you want to change the base?
Conversation
Committed by rlagkswn00 on behalf of kuring.korea@gmail.com
Committed by rlagkswn00 on behalf of kuring.korea@gmail.com
Committed by rlagkswn00 on behalf of kuring.korea@gmail.com
Committed by rlagkswn00 on behalf of kuring.korea@gmail.com
Committed by rlagkswn00 on behalf of kuring.korea@gmail.com
|
사용자 판별은 api key로 하는거 맞나요? 아 액세스토큰인가 |
hwan-cs
left a comment
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.
동아리 즐겨찾기 추가할때는 헤더에 fcm 토큰이나 엑세스 토큰이 필요없는데 즐겨찾기 제거는 엑세스 토큰이 필요한거.. 맞나유?
둘다 fcmToken && accessToken 필요하지 않을까 싶은데요?!
그리고 사소하긴 한데 이런 api의 경우 성공 응답으로 즐겨찾기 수를 반환하는것도 고려 해보는건 어떨까요??
조회 책임에 가깝긴 한데, 더 UX 친화적이기도 합니다ㅎㅎ
Committed by rlagkswn00 on behalf of kuring.korea@gmail.com
Committed by rlagkswn00 on behalf of kuring.korea@gmail.com
…-api-docs into set-clubs-5-apis
zbqmgldjfh
left a comment
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.
어려울수도 있는데, 지금 yaml spec이 좀 각게로 다 분리된거 같은데,
요청 시작부터 반환까지의한 사이클의 API 흐름은 하나의 파일로 보면 어떨까요?
지금은 너무 DTO단위로만 쪼개개둔거라?
고민 정도만 해보고, 반영은 필수는 아닙니다!! 자신의 뜻대로 밀고 나가주시길!~
📝 개요
작업 내용
POST /api/v2/users/bookmarks/clubsDELETE /api/v2/users/bookmarks/clubs변경된 API 명세서 경로
https://ku-ring.stoplight.io/docs/ku-ring/branches/set-clubs-5-apis/6qd89rl96a1he-
🔍 주요 변경 사항
기타