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: 2차 스프린트 기능 구현 #141

Merged
merged 249 commits into from
Dec 15, 2023
Merged

feat: 2차 스프린트 기능 구현 #141

merged 249 commits into from
Dec 15, 2023

Conversation

JeongUijeong
Copy link
Contributor

💡 Motivation

  • 2차 스프린트(추가 기능 및 리팩토링)를 main 브랜치에 적용해야 합니다.

📌 Changes

  • 2차 스프린트 기능 구현

🫱🏻‍🫲🏻 To Reviewers

  • 2차 스프린트 구현이 마무리 되었습니다! 모두 수고하셨습니다!! 😊👍🏻

DT-Career and others added 30 commits November 15, 2023 17:00
refactor: 회원, Refresh 토큰 테이블 속성에 코멘트 추가
refactor: 예약/예약 상품/장바구니/별점 테이블 column comment 추가
refactor: 숙소, 숙소 이미지, 객실 테이블에 comment를 추가합니다.
- Querydsl의 Projections 기능을 사용해 RoomWithProductResponseDto로 바로 조회할 수 있도록 구현
jo0oy and others added 19 commits December 15, 2023 02:35
…oring

refactor: 예약 도메인 리팩토링
# Conflicts:
#	src/main/java/com/fc/shimpyo_be/domain/reservationproduct/dto/request/ReservationProductRequestDto.java
…-room-refactoring

refactor: 예약 상품/객실 도메인 리팩토링
refactor: product - room - cart 내 log.debug 삭제 및 접근 제어자 수정, transaction(readOnly=true)를 통해 성능 향상
docs: 2차 스프린트 개발을 적용한 리드미 수정
@JeongUijeong JeongUijeong added the 🛠️feature 기능 개발 label Dec 15, 2023
@JeongUijeong JeongUijeong self-assigned this Dec 15, 2023
@JeongUijeong JeongUijeong changed the title feat: Shimpyo_BE 2차 스프린트 기능 구현 feat: 2차 스프린트 기능 구현 Dec 15, 2023
Copy link

github-actions bot commented Dec 15, 2023

Test Results

105 tests  +24   105 ✔️ +24   5s ⏱️ -1s
  62 suites +16       0 💤 ±  0 
  62 files   +16       0 ±  0 

Results for commit a06537a. ± Comparison against base commit 7bd8f91.

This pull request removes 6 and adds 30 tests. Note that renamed tests count towards both.
com.fc.shimpyo_be.domain.reservationproduct.docs.ReservationProductRestControllerDocsTest ‑ cancel()는 예약 주문 상품을 취소할 수 있다.
com.fc.shimpyo_be.domain.reservationproduct.unit.controller.ReservationProductRestControllerTest ‑ [api][DELETE][정상] 예약 주문 상품 취소 API 테스트
com.fc.shimpyo_be.domain.reservationproduct.unit.service.ReservationProductServiceTest ‑ 예약 상품 정보가 존재하지 않으면, 예약 상품을 취소할 수 있다.
com.fc.shimpyo_be.domain.reservationproduct.unit.service.ReservationProductServiceTest ‑ 예약 상품 주문자 정보와 현재 인증 객체 정보가 일치하지 않으며, 예약 상품을 취소할 수 없다.
com.fc.shimpyo_be.domain.reservationproduct.unit.service.ReservationProductServiceTest ‑ 예약 상품을 취소할 수 있다.
com.fc.shimpyo_be.domain.star.unit.controller.StarRestControllerTest ‑ [api][POST] 별점 등록 API 성공 테스트
com.fc.shimpyo_be.domain.favorite.docs.FavoriteRestControllerDocsTest ‑ cancel은 즐겨찾기를 취소할 수 있다.
com.fc.shimpyo_be.domain.favorite.docs.FavoriteRestControllerDocsTest ‑ getFavorites()은 즐겨찾기 목록을 조회할 수 있다.
com.fc.shimpyo_be.domain.favorite.docs.FavoriteRestControllerDocsTest ‑ register()은 즐겨찾기를 등록할 수 있다.
com.fc.shimpyo_be.domain.favorite.unit.controller.FavoriteRestControllerTest$Context_cancel ‑ 즐겨찾기를 취소할 수 있다.
com.fc.shimpyo_be.domain.favorite.unit.controller.FavoriteRestControllerTest$Context_getFavorites ‑ 즐겨찾기 목록을 조회할 수 있다.
com.fc.shimpyo_be.domain.favorite.unit.controller.FavoriteRestControllerTest$Context_register ‑ 즐겨찾기를 등록할 수 있다.
com.fc.shimpyo_be.domain.favorite.unit.repository.FavoriteRepositoryTest$Context_findAllByMemberId ‑ 회원으로 즐겨찾기 정보 목록을 조회할 수 있다.
com.fc.shimpyo_be.domain.favorite.unit.repository.FavoriteRepositoryTest$Context_findByMemberAndProduct ‑ 회원과 숙소로 즐겨찾기 정보를 조회할 수 있다.
com.fc.shimpyo_be.domain.favorite.unit.service.FavoriteServiceTest$Context_delete ‑ 즐겨찾기를 취소할 수 있다.
com.fc.shimpyo_be.domain.favorite.unit.service.FavoriteServiceTest$Context_getFavorites ‑ 즐겨찾기 목록을 조회할 수 있다.
…

♻️ This comment has been updated with latest results.

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.

2주동안 우리 조 너무너무 고마웠고 정말정말 많이 수고했어요! 이멤버 리멤버❤️❤️❤️

@JeongUijeong JeongUijeong merged commit 108425e into main Dec 15, 2023
5 checks passed
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.

4 participants