Skip to content
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 #286

Merged
merged 2 commits into from
Oct 22, 2024
Merged

Feat: 북마크 여부 확인 API #286

merged 2 commits into from
Oct 22, 2024

Conversation

muncool39
Copy link
Member

Summary

현재 로그인한 유저가 파라미터와 일치하는 리소스에 북마크 했는지 확인하는 API를 개발했습니다.

request

  • GET
  • /bookmark

param

  • type (EVENT/BOOTH)
  • resourceId

파라미터를 모두 입력하지 않을 경우 400코드와 함께 오류메세지를 반환합니다.

Key Changes

기존 BookmarkService 에서 유저의 북마크 여부 확인하는 로직을 다른 메서드로 분리해서 사용

Testing

성공 (북마크한 경우)
GET {{local}}/bookmark?type=EVENT&resourceId=70
image

성공 (북마크하지 않은 경우)
GET {{local}}/bookmark?type=BOOTH&resourceId=72
image

To Reviewers

  • 행사 부스 조회가 로그인 없이도 돼서 북마크 여부를 조회 결과에 포함시키기 복잡해 api를 따로 분리했습니다.
    매니저 여부 확인하듯이 프론트 쪽에서 토큰 정보가 null 이 아닌 경우 해당 api를 호출하도록 하면 될 것 같은데 해당 내용은 회의 때 전달하도록 하겠습니다!
  • 궁금하신 점, 개선할 점 등 편하게 의견 주세요! 😃

@muncool39 muncool39 added the feature 기능 개발과 관련된 내용입니다. label Oct 21, 2024
@muncool39 muncool39 requested a review from gitseoyeon October 21, 2024 03:11
@muncool39 muncool39 self-assigned this Oct 21, 2024
Copy link
Member

@gitseoyeon gitseoyeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다~ 고생하셨어요!!

@muncool39 muncool39 merged commit 5a4a5af into dev Oct 22, 2024
3 checks passed
@muncool39 muncool39 deleted the feat/#280 branch October 22, 2024 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 개발과 관련된 내용입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 북마크 여부 확인 기능
2 participants