modern-agile-team / Menbosha-back

모던애자일 6기 멘보샤 백엔드입니다
MIT License
4 stars 0 forks source link

Setting/#316/restrict varchar length #324

Closed hobiJeong closed 6 months ago

hobiJeong commented 6 months ago

Description

intergrate migration

테이블 만들고, 자잘한 수정들이 있었던 migration 파일들을 한 파일들로 묶었습니다.

modify table and column

몇몇의 테이블들 및 칼럼들의 이름을 좀 더 자연스럽게 수정했고 length들의 제한도 줬습니다.

modify code

테이블 변경점들에 따라 수정돼야 할 코드들도 수정 했습니다. PR크기가 굉장히 커서 리뷰가 힘드실 것 같아서 일단 죄송합니다..

하지만 이 과정에서 꼼꼼하게 살피긴 했는데 혹시 에러가 발생할 수도 있는 코드들도 있을 수 있습니다. 꼼꼼하게 같이 리뷰해주셔야 합니다 부탁드리겠습니다ㅠㅠ

To Reviewer

아마 migration관련 파일들은 안보셔도 될 것 같고, 대신 entity 파일들을 보면 될 것 같습니다. 해당 피알 머지되면 변경된 스키마에 따라 애플리케이션 코드 상에서도 제한을 두는 코드를 추가하겠습니다.

Reference Link

Related Issue Link

316

2swo commented 6 months ago

엔티티 변경점 위주로 전부 확인했고, 경로수정 관련된거는 빠르게 넘겼습니다. 일단 확인은 다 했는데 추후 혹시라도 문제 있다면 발견 시 바로 해결하는 방향으로 가면 될 것 같습니다. (문제 없을듯?)

hobiJeong commented 6 months ago

엔티티 변경점 위주로 전부 확인했고, 경로수정 관련된거는 빠르게 넘겼습니다. 일단 확인은 다 했는데 추후 혹시라도 문제 있다면 발견 시 바로 해결하는 방향으로 가면 될 것 같습니다. (문제 없을듯?)

꼼꼼하게 봐주셔서 감사합니다   🙇 🙇