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

feat: 온보딩 정보를 저장, 조회하는 api를 추가한다. #19

Merged
merged 4 commits into from
Aug 6, 2024

Conversation

rlarltj
Copy link
Member

@rlarltj rlarltj commented Aug 5, 2024

📄 변경 사항

  • 온보딩 정보를 저장하는 api를 추가합니다.
  • 온보딩 여부를 조회하는 api를 추가합니다.

@rlarltj rlarltj requested a review from min429 August 5, 2024 15:04
@rlarltj rlarltj self-assigned this Aug 5, 2024
@min429
Copy link
Collaborator

min429 commented Aug 5, 2024

@rlarltj 기서님!
혹시 UserProfile이랑 User랑 일대일 매핑인거같은데 UserProfile 필드인 userId에 어노테이션이 붙어야 하지 않나요??

@rlarltj
Copy link
Member Author

rlarltj commented Aug 6, 2024

@min429 논리적으로는 1:1 관계가 맞습니다! 만,, 저는 연관관계 매핑을 선호하지 않습니다..
나중에 추가도 가능하고요 ㅎㅎ

요런 영상 몇 개 보시면 도움이 되실 것 같습니다!!

@rlarltj rlarltj merged commit b397ebf into main Aug 6, 2024
1 check passed
@rlarltj rlarltj deleted the ISSUE-110 branch August 6, 2024 13:51
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