manito42 / backend

42 Manito Backend Repository
https://42manito.com
3 stars 2 forks source link

[Feature] profile description scheme 변경 #95

Closed koreanddinghwan closed 1 year ago

koreanddinghwan commented 1 year ago

Description

https://github.com/orgs/manito42/discussions/133#discussion-5638282 에서 언급된 API수정사항입니다.

Todo


etc

1.

dto의 minlength 0으로 변경.


문제상황

제가 알기론, database에서 NULL의 사용을 지양해야하는 이유가, 값이 상태가 애매모호 하기때문으로 알고 있습니다.
저희의 경우에는 NULL이 초기상태인지/사용자가 입력 빈 상태를 원하는건지/NULL과 ""의 의미 차이가 모호하기 때문인 것 같습니다.

해결



2.

scheme의 default value를 ''로 변경.

문제

TEXT에 default value할당불가

Error: P3018

A migration failed to apply. New migrations cannot be applied before the error is recovered from. Read more about how to resolve migration issues in a production database: https://pris.ly/d/migrate-resolve

Migration name: 20230917160953_feature_change_profile_description_95

Database error code: 1101

Database error:
BLOB, TEXT, GEOMETRY or JSON column 'description' can't have a default value

Please check the query number 1 from the migration file.

관련 글

https://stackoverflow.com/questions/3466872/why-cant-a-text-column-have-a-default-value-in-mysql

해결