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

refactor: 포인트와 숙소 가독성 개선 #225

Merged
merged 7 commits into from
Feb 1, 2024

Conversation

decten
Copy link
Contributor

@decten decten commented Jan 30, 2024

💡Motivation

  • 숙소와 쿠폰의 메서드 중복 코드 제거 및 단순화

📌Changes

🫱🏻‍🫲🏻To Reviewers

image

지금 제가 개발한 api에서 couponName까지 제대로 나오고 있는데
https://fast-catch-front-end-gold.vercel.app/
홈페이지에서는 쿠폰 정보가 안 나오고 있습니다, 쿠폰 값이 이상해서 프런트에서 뺐는지 아니면 서버에서 못 받고 있는지 내일 여쭤보려고 합니다
-> 쿠폰 테이블에 데이터 추가함으로써 해결(미니프 숙소는 쿠폰 테이블만 접근하고 coupon-issuance 사용 안 함)

  • 테스트 코드 작성하고 있는데 쉽지 않네요ㅠㅠ 이번주 중으로 올릴 수 있도록 노력하겠습니다
  • 테스트 코드 이후에 포인트에도 팀장님처럼 CQRS 패턴 적용 해보겠습니다

# Conflicts:
#	src/main/java/com/backoffice/upjuyanolja/domain/accommodation/service/AccommodationCommandService.java
#	src/main/java/com/backoffice/upjuyanolja/domain/point/service/PointService.java
#	src/main/java/com/backoffice/upjuyanolja/domain/room/service/RoomCommandService.java
#	src/test/java/com/backoffice/upjuyanolja/domain/room/unit/service/RoomCommandServiceTest.java
@decten decten added the 🚧refactoring 리팩토링 label Jan 30, 2024
@decten decten self-assigned this Jan 30, 2024
@decten decten changed the title refactor: 포인트 & 숙소 refactor: 포인트와 숙소 가독성 개선 Jan 30, 2024
@decten decten added the 👁️‍🗨️revision 개선 필요 label Jan 30, 2024
Copy link

github-actions bot commented Jan 30, 2024

Test Results

100 tests   94 ✅  5s ⏱️
 61 suites   6 💤
 61 files     0 ❌

Results for commit 8192642.

♻️ This comment has been updated with latest results.

@decten decten removed the request for review from gahyuun January 30, 2024 14:51
Copy link
Contributor

@matrixpower1004 matrixpower1004 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

@decten decten merged commit 5dd90be into develop Feb 1, 2024
3 checks passed
@decten decten deleted the feature/refactorPointAccomo branch February 1, 2024 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

3 participants