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

✨ [STMT-200] 스터디 도메인 구현 #92

Merged
merged 13 commits into from
Mar 11, 2024
Merged

✨ [STMT-200] 스터디 도메인 구현 #92

merged 13 commits into from
Mar 11, 2024

Conversation

05AM
Copy link
Member

@05AM 05AM commented Mar 7, 2024

💁 해결 하려는 문제를 적어주세요

  • 스터디 도메인 구현

🤔 어떤 방식으로 해결했는지 적어주세요

  • 스터디 도메인과 JPA 엔티티, 간단한 query usecase와 service를 구현하였습니다.

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

  • 기존 코드를 참고하며 작성했는데 편하게 의견 주시면 감사하겠습니다!
  • 그리고 ApiResponse에서 코드의 통일성을 위해 ErrorCodeSuccessCode를 인수로 받지 않는 메서드를 사용하는 코드를 수정해주시길 부탁 드립니다.
  • flyway의 사용법에 대해 혼란이 있어 sql은 수정하지 않았는데 다음 사항에 답변해주시면 감사하겠습니다.
    • V1_init.sql 에 DDL을 추가해야 하는지, 따로 V1.n 파일을 만들어야 하는지 궁금합니다.
    • 개발의 편의성을 위해 스터디 dummy data도 만들어둘까 고민 중인데 어떻게 생각하시는지 의견을 듣고 싶습니다.
    • 스터디 분야에 대한 insert 문은 기획에서 결정되는 대로 작성하겠습니다!

=> 이후 flyway에 대해 가볍게나마 공부해볼 예정입니다. 너그러이 봐주시면 감사하겠습니다 :)

🧑‍🏫 이해를 위해 필요한 자료가 있다면 첨부해주세요

image

@05AM 05AM added the D-1 1일 전 까지 리뷰해주세요 label Mar 7, 2024
@05AM 05AM self-assigned this Mar 7, 2024
@05AM 05AM requested a review from zxcv9203 as a code owner March 7, 2024 15:26
Copy link
Collaborator

@zxcv9203 zxcv9203 left a comment

Choose a reason for hiding this comment

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

스터디 도메인 작성 고생하셨습니다. :)
코멘트 확인 부탁드립니다.

@zxcv9203
Copy link
Collaborator

zxcv9203 commented Mar 8, 2024

  • V1_init.sql 에 DDL을 추가해야 하는지, 따로 V1.n 파일을 만들어야 하는지 궁금합니다.

추가로 만들어주시면 됩니다. V1은 이미 생성된 sql이기 때문에 정상적으로 인식되지 않게 됩니다.

  • 개발의 편의성을 위해 스터디 dummy data도 만들어둘까 고민 중인데 어떻게 생각하시는지 의견을 듣고 싶습니다.

넵 좋은 것 같습니다. 어제 말씀드렸던 sql 파일에 추가해주시면 좋을 것 같아요 👍

@github-actions github-actions bot added D-0 바로 리뷰가 필요해요 and removed D-1 1일 전 까지 리뷰해주세요 labels Mar 8, 2024
@05AM 05AM requested a review from zxcv9203 March 9, 2024 08:57
Copy link
Collaborator

@zxcv9203 zxcv9203 left a comment

Choose a reason for hiding this comment

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

flyway 파일 버전 변경 부탁드립니다 🙇

src/main/resources/db/migration/V1.3__add_study_table.sql Outdated Show resolved Hide resolved
@05AM 05AM merged commit fe1b138 into dev Mar 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-0 바로 리뷰가 필요해요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants