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

[Feature] 이동봉사자, 이동봉사 중개 Entity 분리 #15

Merged
merged 3 commits into from
Oct 27, 2023

Conversation

kyeong-hyeok
Copy link
Member

💡 연관된 이슈

close #13

📝 작업 내용

  • 이동봉사자, 이동봉사 중개 Entity 분리
  • 이동봉사자, 이동봉사 중개 Entity 변경 적용

💬 리뷰 요구 사항

@kyeong-hyeok kyeong-hyeok added ✨ Feature 기능 개발 Priority: Medium 우선순위 중간 🐯 Koeyhk 담당자 labels Oct 27, 2023
@kyeong-hyeok kyeong-hyeok linked an issue Oct 27, 2023 that may be closed by this pull request
2 tasks
Copy link
Member

@hojeong2747 hojeong2747 left a comment

Choose a reason for hiding this comment

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

이동봉사자와 중개 Entity를 분리하기까지 많은 고민을 해주어서 고맙습니다! 의미와 관계가 명확히 드러나서 api 개발할 때 수월할 것 같아요👍

private String password; // 비밀번호
@Column(length = 20, nullable = false)
private String name; // 중개자 이름/중개 단체명
@Column(columnDefinition = "TEXT")
Copy link
Member

Choose a reason for hiding this comment

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

우와 columnDefinition에 대해서 처음 알았어요.
해당 컬럼이 데이터베이스에 어떻게 생성될 것인지를 직접 지정할 수 있는 어노테이션이고, "TEXT"는 대용량의 문자열을 저장할 수 있는 데이터 타입을 의미하는 것 맞나요?

Copy link
Member Author

@kyeong-hyeok kyeong-hyeok Oct 27, 2023

Choose a reason for hiding this comment

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

맞습니다! 근데 columnDefinition = "TEXT"는 String으로 지정했을 때 varchar(255) 제한을 해제하는 건데, 이미지 url에는 사용을 안 해도 될 것 같아서 지우는 게 좋을 것 같아요! (보통 게시판 글의 내용의 제한을 해제할 때 썼습니다!)
이름의 length는 20으로 지정해두고 다른 컬럼은 따로 지정 안 했는데, 이 부분은 자세한 정보까지 나오면 바꿀게요!

@kyeong-hyeok kyeong-hyeok merged commit 2812f46 into develop Oct 27, 2023
1 check passed
@kyeong-hyeok kyeong-hyeok deleted the feat/13-entity-divide branch October 27, 2023 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🐯 Koeyhk 담당자 Priority: Medium 우선순위 중간
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 이동봉사자, 이동봉사 중개 Entity 분리
2 participants