-
Notifications
You must be signed in to change notification settings - Fork 1
교수 소속(department) 추가 #384
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
교수 소속(department) 추가 #384
Conversation
| ): ProfessorLanguagesDto { | ||
| val koreanProfessorDto = createProfessor(LanguageType.KO, req.ko, mainImage) | ||
| val englishProfessorDto = createProfessor(LanguageType.EN, req.ko, mainImage) | ||
| val englishProfessorDto = createProfessor(LanguageType.EN, req.en, mainImage) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹시 원래 코드에 의도가 있었을까요?
잘 모르겠어 제가 맞다 생각하는 대로 바꿔보았습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 이건 몰랐네 ㅋㅋ 굿입니당
| @@ -0,0 +1,2 @@ | |||
| ALTER TABLE professor | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 돌리면 모두 null로 들어있을텐데 추후 마이그레이션 등으로 채울 예정이겠죠?
- nullable한 필드가 아니라면 empty string을 default 값으로 하는 non null 필드로 바꾸는 것도 좋을듯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이미 여러 소속 교수님들이 등록되어 계셔서 저희가 특정 값으로 채워 넣는 것은 힘들 것 같다 생각했어요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
empty string 쓰는 걸로 바꾸는 게 좋을 것 같습니다! 곧 수정하겠습니다
교수 소속(department) 추가
한줄 요약
String type 필드 하나 추가했습니다
null 가능하게 만들었고, 아래처럼 요청할 수 있습니다