Skip to content

Conversation

@rlagkswn00
Copy link
Member

@rlagkswn00 rlagkswn00 commented Jan 26, 2026

📝 개요

작업 내용

POST /api/v2/users/bookmarks/clubs
image

DELETE /api/v2/users/bookmarks/clubs
image

변경된 API 명세서 경로

https://ku-ring.stoplight.io/docs/ku-ring/branches/set-clubs-5-apis/6qd89rl96a1he-

  • Draft 태그 내 두가지 항목

🔍 주요 변경 사항

  • 사용자 동아리 즐겨찾기 추가, 삭제 API를 추가했습니다.

기타

  • 처음 작업하는거라 혹시 이상하거나 요상한게 있다면 이야기 해주세여(꾸벅)

kuring.korea@gmail.com added 5 commits January 26, 2026 15:50
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
@rlagkswn00 rlagkswn00 self-assigned this Jan 26, 2026
@rlagkswn00 rlagkswn00 added the documentation Improvements or additions to documentation label Jan 26, 2026
@mwy3055
Copy link
Member

mwy3055 commented Jan 27, 2026

사용자 판별은 api key로 하는거 맞나요? 아 액세스토큰인가

Copy link

@hwan-cs hwan-cs left a 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 친화적이기도 합니다ㅎㅎ

kuring.korea@gmail.com and others added 7 commits January 29, 2026 10:41
Copy link
Member

@zbqmgldjfh zbqmgldjfh left a 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단위로만 쪼개개둔거라?

고민 정도만 해보고, 반영은 필수는 아닙니다!! 자신의 뜻대로 밀고 나가주시길!~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants