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

3단계 - 수강신청(DB 적용) #515

Open
wants to merge 5 commits into
base: movingone
Choose a base branch
from

Conversation

movingone
Copy link

@movingone movingone commented Apr 18, 2024

2단계에서 주신 수정사항만 리팩토링 했습니다

'앞 단계에서 구현한 도메인 모델을 DB 테이블과 매핑하고, 데이터를 저장한다'

라는 요구사항 이해를 못했습니다. 다른 분들의 3단계 코드를 계속 봤는데. 해보질 않아서 그런지 이해가 어렵습니다. 어떤식으로 학습해서 접근해야 할까요

@neojjc2
Copy link

neojjc2 commented Apr 18, 2024

@movingone 동원님 제가 리뷰했던 리뷰이중에 3단계를 정말 잘 해주신 분이 계신데요, 방향잡기 어려우시다면
요 내용 한번 참고하셔서 따라해보시는것도 괜찮을 것 같습니다 🙇
참고 부탁 드리겠습니다 🙏

#434

Copy link

@neojjc2 neojjc2 left a comment

Choose a reason for hiding this comment

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

안녕하세요 동원님 🙇

3단계 참고하실만한 내용 코멘트 남겼습니다.
한번 참고해보시고 진행해주시면 될 것 같습니다 😄

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