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: erd 변경과 코드 수정 #227

Merged
merged 21 commits into from
Feb 7, 2024
Merged

refactor: erd 변경과 코드 수정 #227

merged 21 commits into from
Feb 7, 2024

Conversation

decten
Copy link
Contributor

@decten decten commented Feb 3, 2024

💡Motivation

  • 개발 피드백을 바탕으로 db erd를 수정하고, 코드를 변경했습니다
    https://www.erdcloud.com/d/fpNBFtPodeq6TPi76
  • 테스트 코드에서 자카르타 transactional로 된 부분을 springframework import로 수정 해드렸습니다

📌Changes

  • address 테이블 삭제 및 accommodation에 병합
  • accommodation & room 모두 option의 외래 키 관계 변경
  • roomStock 양방향 삭제
  • accommodation & room 모두 image 양방향 삭제
  • room에서 roomPrice와 외래 키 관계 변경

🫱🏻‍🫲🏻To Reviewers

  • 테스트 코드는 모두 통과 했고 채원님이랑 지상님 서비스 코드는 거의 변동이 없지만, api가 원래처럼 잘 동작하는지 다들 확인 부탁 드립니다
    카톡으로 공유 드렸던 것처럼 지상님은 쿼리 dsl에서 필드가 삭제 됨에 따라 수정된 부분이 있어 확인 부탁드립니다
  • 특히 팀장님 부분이 많이 변경 됐는데, 오픈 api와 이미지 로직도 괜찮은지 확인 부탁 드립니다
  • 지금 pr 보시다시피 하나 건드리면 다른 도메인 비즈니스 코드 & 테스트 코드 & sql 작성에 모두 영향이 가기 때문에, 수정 사항이 있다면 해당 브랜치로 커밋을 보내주시고 지금 pr로 한 번에 develop에 보내려고 합니다
  • 해당 pr은 모든 분들이 괜찮으시고, sql 백업까지 제가 다 준비 하면 카톡으로 한 번 더 말씀 드리고 제가 머지하겠습니다

decten added 16 commits February 2, 2024 01:14
…한 메서드 추가 및 기존 메서드의 파라미터 변경
@decten decten added the 🚧refactoring 리팩토링 label Feb 3, 2024
@decten decten self-assigned this Feb 3, 2024
Copy link

github-actions bot commented Feb 3, 2024

Test Results

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

Results for commit 9b62002.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@JeongUijeong JeongUijeong left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!! 제가 수정할 부분 수정한 후 approve 하도록 하겠습니다! 😊

Copy link
Contributor

@JeongUijeong JeongUijeong 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

@chaewon12 chaewon12 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

@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.

브랜치 pull 받아서 확인하였고 현재까지는 에러 없습니다. 수고하셨습니다. 🌹👍

@decten decten merged commit 3d21159 into develop Feb 7, 2024
3 checks passed
@decten decten deleted the feature/refactorDB branch February 7, 2024 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚧refactoring 리팩토링
Projects
Development

Successfully merging this pull request may close these issues.

4 participants