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

[1 - 2단계 방탈출 예약 대기] 페드로(류형욱) 미션 제출합니다. #78

Merged
merged 45 commits into from
May 18, 2024

Conversation

hw0603
Copy link
Member

@hw0603 hw0603 commented May 16, 2024

안녕하세요 미르! BE 6기 페드로입니다.

이번 미션 시작 당시 저와 페어 모두 직전 미션이 완료되지 않은 상태라, 우선 머지되지 않은 상태였던 페어의 코드로 이번 미션을 진행했습니다.
직전 미션의 요구사항이였던 인터셉터나 JWT 인증 관련 부분은 이번 미션의 초점은 아니라고 생각하여 아직 리팩토링되지 않은 상태입니다!

현재 지식 수준

스프링이나 JPA를 공부하면서 사용하는 것은 사실상 처음이고, Python 기반의 간단한 웹 서버를 만들어 본 경험은 있습니다.
관계형 데이터베이스를 공부한 적은 있지만 아직 깊게 이해하지는 못했습니다.

궁금한 부분

양방향 관계와 단방향 관계를 결정하는 명확한 기준을 세우지 못한 것 같아요. 학습할 키워드나 힌트를 조금 받을 수 있을까요?

이번 미션에서 변경된 부분은 이곳에서 확인하실 수 있습니다.

리뷰 잘 부탁드려요! 😀

hw0603 added 30 commits May 14, 2024 20:41
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
- jdbc -> data-jpa

Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
- Schedule 테이블 분리

Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
hw0603 added 11 commits May 16, 2024 11:31
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
경
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
@hw0603 hw0603 marked this pull request as draft May 16, 2024 08:03
Co-authored-By <2001yerin@naver.com>
@hw0603 hw0603 marked this pull request as ready for review May 16, 2024 08:41
Copy link

@ddu0422 ddu0422 left a comment

Choose a reason for hiding this comment

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

안녕하세요 페드로! 미르입니다 :)

JPA로 변경 잘 해주셨네요~
몇 가지 코멘트 남겼으니 확인 부탁드립니다!

궁금한 점은 코멘트 남겨주시거나 언제든지 DM주세요 😄

hw0603 added 3 commits May 17, 2024 14:14
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
Co-authored-By <2001yerin@naver.com>
@hw0603
Copy link
Member Author

hw0603 commented May 17, 2024

안녕하세요! 리뷰를 반영 하고 보니 수정한 부분이 거의 없다시피 하네요..😂
짚어주신 부분들 코멘트와 질문들 남겨 두었습니다! 감사합니다😊

Copy link

@ddu0422 ddu0422 left a comment

Choose a reason for hiding this comment

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

안녕하세요 페드로! 미르입니다 :)

JPA의 기본 내용은 충분히 이해했다고 생각하여 머지를 진행하겠습니다.
몇 가지 코멘트는 다음 미션을 진행하며 고민해보시면 어떨까합니다 ㅎ_ㅎ
다음 미션도 화이팅입니다!

궁금한 내용이 있으면 코멘트 남겨주시거나 언제든지 DM주세요~

@ddu0422 ddu0422 merged commit 53fa974 into woowacourse:hw0603 May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants