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] 북마크 기능 추가 및 변수명 변경 #233

Merged
merged 5 commits into from
Dec 30, 2024
Merged

Conversation

jung2941
Copy link
Collaborator

@jung2941 jung2941 commented Dec 30, 2024

변경 사항

  • 북마크 기능 추가
  • 변수명 변경
    response.data?.message === "USER_200"
    message -> code로 변경

테스트 결과 (테스트를 했으면)

ex) 베이스 브랜치에 포함되기 위한 코드는 모두 정상적으로 동작해야 합니다. 결과물에 대한 스크린샷, GIF, 혹은 라이브 데모가 가능하도록 샘플API를 첨부할 수도 있습니다.
image

@jung2941 jung2941 added the 🏷️ Fix 버그 수정 label Dec 30, 2024
@jung2941 jung2941 self-assigned this Dec 30, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Code review by ChatGPT

@jung2941 jung2941 changed the title [fix] 변수명 변경 [fix] 북마크 기능 추가 및 변수명 변경 Dec 30, 2024
@jung2941 jung2941 changed the title [fix] 북마크 기능 추가 및 변수명 변경 [feat] 북마크 기능 추가 및 변수명 변경 Dec 30, 2024
BellYun
BellYun previously approved these changes Dec 30, 2024
Copy link
Contributor

@BellYun BellYun left a comment

Choose a reason for hiding this comment

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

빠른 수정 좋습니다! 수고하셨습니다

Copy link
Contributor

@BellYun BellYun left a comment

Choose a reason for hiding this comment

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

빠른 conflict 해결까지 좋습니다!

@jung2941 jung2941 merged commit aaf5828 into develop Dec 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ Fix 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants