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

# 127 Certification 도메인 개발 #128

Merged
merged 4 commits into from
Nov 13, 2023
Merged

Conversation

JuuuuHong
Copy link
Collaborator

💡 개요

Certification 도메인 개발

📃 작업내용

Certification Entity 생성
Certification Repository 생성

@JuuuuHong JuuuuHong self-assigned this Nov 13, 2023
@JuuuuHong JuuuuHong linked an issue Nov 13, 2023 that may be closed by this pull request
@Column(columnDefinition = "VARCHAR(30)", nullable = false)
val name: String,

@Column(columnDefinition = "DATETIME(6)", nullable = false)
Copy link
Collaborator

Choose a reason for hiding this comment

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

학생활동 파트를 이미 끝내버리고 나서 든 생각인데 활동 날짜와 같이 이것도 취득일자 이기 때문에 정확한 시간이 필요하지 않아보여요 그래서 DATE 데이터 타입을 사용하는걸 고려해보는게 어떨까요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

d2f1376 LocalDate 로 적용해봤습니다!

Comment on lines 19 to 21
@ManyToOne(fetch = FetchType.LAZY)
@JoinColumn(name = "user_id", columnDefinition = "BINARY(16)")
val user: User,
Copy link
Contributor

Choose a reason for hiding this comment

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

userId만 갖고있어도 괜찮지 않을까요?

Copy link
Contributor

@esperar esperar Nov 13, 2023

Choose a reason for hiding this comment

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

ERD에서는 연관관계가 student랑 되어있습니다 헷갈리신거같은데 수정 부탁드려요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

7a9173c 수정했습니다!

@JuuuuHong JuuuuHong merged commit c646ad9 into master Nov 13, 2023
1 check passed
@esperar esperar deleted the 127-create/certificate branch November 20, 2023 00:25
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.

자격증 entity 및 repository 생성
3 participants