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

[#16] Task CRUD DB 접근 로직 구현 및 core, storage 병합 #18

Merged
merged 8 commits into from
Sep 29, 2024

Conversation

olivejua
Copy link
Collaborator

@olivejua olivejua commented Sep 23, 2024

변경 내용

  • storage 모듈 삭제하고, core에 DB 설정과 의존성 추가
  • create_table.sql 파일에 database 생성, task 테이블 생성 쿼리 추가
  • TaskEntity, TaskRepositoroy 별도로 생성함 (이 부분은 필요시 추후 분리할 수도 있음)
  • application.yml 설정파일에 core 모듈의 application-storage.yml 파일 적용

추가 내용

@olivejua olivejua self-assigned this Sep 23, 2024
@olivejua olivejua added the feature New feature or request label Sep 23, 2024
Copy link
Collaborator

@f-lab-lyan f-lab-lyan left a comment

Choose a reason for hiding this comment

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

전체적으로 문제는 없는 것 같습니다. 다만, DefaultTaskRepository, TaskJpaRepository, TaskRepository의 관계에 대해서 고민을 해보면 좋을 것 같습니다. 나머지는 괜찮습니다. 그리고 다음 스텝으로는 "테스트 환경에서도 h2가 아니라, mysql을 쓰려면 어떻게 하는것이 좋을까?"에 대해서 고민해보시면 좋을 것 같습니다. 고생하셨습니다.

@olivejua
Copy link
Collaborator Author

넵 Repository 관계와 테스트환경에서의 DB 접근 방법에 대해 고민해보고 이 부분 추가 PR 제출하겠습니다. 리뷰 감사합니다. 🙏

@olivejua olivejua merged commit 88c6c6f into main Sep 29, 2024
1 check passed
@olivejua olivejua deleted the feature/16-task-crud-data branch October 6, 2024 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants