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.

Sign up for GitHub

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

Jump to bottom

feat: 즐겨찾기 기능 구현 #116

Merged
merged 33 commits into from
Dec 12, 2023
Merged

feat: 즐겨찾기 기능 구현 #116

merged 33 commits into from
Dec 12, 2023

Conversation

JeongUijeong
Copy link
Contributor

💡 Motivation

  • 회원이 숙소를 즐겨찾기에 등록, 조회, 취소할 수 있도록 기능을 구현해야 합니다.

📌 Changes

  • 즐겨찾기 등록 기능 구현
  • 즐겨찾기 등록 기능 구현
  • 즐겨찾기 목록 조회 기능 구현
    • 페이지네이션 추가
  • 즐겨찾기 취소 기능 구현
  • 서비스 테스트 코드 작성
  • 레포지토리 테스트 코드 작성
  • 컨트롤러 테스트 코드 작성
  • REST Docs 테스트 코드 작성
  • adoc 파일 작성

🫱🏻‍🫲🏻 To Reviewers

  • 즐겨찾기 기능 구현 완료했습니다! 재철님은 해당 부분 적용하여 숙소 조회 시 즐겨찾기 여부를 응답할 수 있도록 수정해주시면 감사하겠습니다! 😊👍🏻🩷

this close #113

@JeongUijeong
#113 feat: 즐겨찾기 관련 에러코드 추가
bff9b88
@JeongUijeong
#113 feat: 이미 즐겨찾기 등록한 숙소에 대한 등록 예외 생성
f6bb04a
@JeongUijeong
#113 feat: 즐겨찾기 응답 DTO 생성
9ad0dbd
@JeongUijeong
#113 feat: 회원, 숙소로 즐겨찾기 내역 조회 메서드 추가
ef25a2b
@JeongUijeong
#113 feat: 즐겨찾기 등록 기능 구현
4c8e882
@JeongUijeong
#113 feat: 즐겨찾기 등록 API 구현
4b6de65
@JeongUijeong
#113 feat: 회원으로 즐겨찾기 목록 조회 메서드 추가
b9c5d1a
@JeongUijeong
#113 feat: 즐겨찾기 조회 기능 구현
aa30b7b
@JeongUijeong
#113 feat: 즐겨찾기 목록 조회 API 구현
c62dfb0
@JeongUijeong
#113 refactor: 즐겨찾기 예외 코드 수정
724fcd7
@JeongUijeong
#113 feat: 즐겨찾기 조회 실패 익셉션 생성
e1e1ed9
@JeongUijeong
#113 feat: 즐겨찾기 삭제 기능 구현
ad63211
@JeongUijeong
#113 refactor: 즐겨찾기 삭제 메서드명 수정
37a2142
@JeongUijeong
#113 feat: 즐겨찾기 취소 API 구현
d0fef4f
@JeongUijeong
#113 fix: 조건문 에러 수정
61b7cda
@JeongUijeong
#113 test: 즐겨찾기 서비스 테스트 코드 작성
a1bf2fe
@JeongUijeong
#113 refactor: 회원을 통한 즐겨찾기 목록 조회 메서드명 수정
0f59c07
@JeongUijeong
#113 refactor: 회원을 통한 즐겨찾기 목록 조회 메서드명 수정 적용
e2caefe
@JeongUijeong
#113 test: 회원을 통한 즐겨찾기 목록 조회 메서드명 수정 적용
3c77845
@JeongUijeong
#113 test: 즐겨찾기 레포지토리 테스트 코드 작성
11b7aa5
@JeongUijeong
#113 feat: 즐겨찾기 커스텀 인터페이스 인터페이스 생성
3391f34
@JeongUijeong
#113 feat: 즐겨찾기 커스텀 레포지토리 구현체 구현
09e8733
@JeongUijeong
#113 feat: 페이징 기능 추가된 즐겨찾기 목록 조회 응답 DTO 생성
0cea8f4
@JeongUijeong
#113 refactor: 즐겨찾기 커스텀 레포지토리를 상속 받도록 수정
b759dde
@JeongUijeong
#113 feat: 즐겨찾기 목록 조회 서비스에 페이징 기능 추가
c629aef
@JeongUijeong
#113 feat: 즐겨찾기 목록 조회 API에 페이징 기능 추가
dcfb89b
@JeongUijeong
#113 test: 즐겨찾기 목록 조회 페이징 기능 추가 적용 수정
b0b739c
@JeongUijeong
#113 test: 즐겨찾기 목록 조회 페이징 기능 추가 적용 수정
e1f3c7e
@JeongUijeong
#113 test: 즐겨찾기 컨트롤러 테스트 작성
5cb315e
@JeongUijeong
#113 test: 즐겨찾기 API 문서화를 위한 테스트 코드 작성
4f2b467
@JeongUijeong JeongUijeong added 📚document 문서 작업 🛠️feature 기능 개발 ✅Test test 관련 labels Dec 12, 2023
@JeongUijeong JeongUijeong self-assigned this Dec 12, 2023
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 12, 2023 •
edited
Loading

Test Results

98 tests  +11   98 ✔️ +11   5s ⏱️ ±0s
62 suites +11     0 💤 ±  0 
62 files   +11     0 ❌ ±  0 

Results for commit 8dbcea1. ± Comparison against base commit 5c99a44.

♻️ This comment has been updated with latest results.

jo0oy
jo0oy approved these changes Dec 12, 2023
View reviewed changes
Copy link
Contributor

@jo0oy jo0oy left a comment

Choose a reason for hiding this comment

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

너무너무 수고했어요~~~❤️👍🏻

wocjf0513
wocjf0513 approved these changes Dec 12, 2023
View reviewed changes
Copy link
Contributor

@wocjf0513 wocjf0513 left a comment

Choose a reason for hiding this comment

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

고생했습니다!!

@JeongUijeong JeongUijeong merged commit 4e16e10 into develop Dec 12, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wocjf0513 wocjf0513 wocjf0513 approved these changes

@jo0oy jo0oy jo0oy approved these changes

Assignees

@JeongUijeong JeongUijeong

Labels
📚document 문서 작업 🛠️feature 기능 개발 ✅Test test 관련
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JeongUijeong @jo0oy @wocjf0513