search

GDSC-University-of-Seoul / 2021-spring-project

Repository for DSC-UOS Spring Project Team
8 stars 0 forks source link

#KDSK-88 / BE 기능 추가를 위한 DB 모델링 수정 #71

Closed rxdcxdrnine closed 3 years ago

rxdcxdrnine commented 3 years ago

기능 Issue

Linked Jira backlog : #KDSK-88

PR 목적

작업 설명

  1. CCTV 테이블 CCTV_MAC 컬럼의 속성을 unique 로 변경
  2. LOG 테이블 추가

이외에 현재 develop 브랜치에서 필요없는 파일/코드를 삭제하였습니다.

라벨링 확인

작업 우선순위 : 상

YubeenSon commented 3 years ago

창구님 모델을 db에 적용시키는 것은 sync - force 쓰는 것 생각하고 계신가요?

rxdcxdrnine commented 3 years ago

창구님 모델을 db에 적용시키는 것은 sync - force 쓰는 것 생각하고 계신가요?

현재 유빈님이 관리하시는 kidskeeper DB 에는 데이터가 담겨있으니 force: true 속성을 사용하시면 안될거에요! 대신 제가 관리하는 local DB 나 AWS RDB 의 kidskeeper_dev DB 의 경우 force:true 속성을 써서 DB 를 초기화시키거나, alter: true 속성으로 모델링 변경 사항만 반영하고는 있습니다.

YubeenSon commented 3 years ago

창구님 모델을 db에 적용시키는 것은 sync - force 쓰는 것 생각하고 계신가요?

현재 유빈님이 관리하시는 kidskeeper DB 에는 데이터가 담겨있으니 force: true 속성을 사용하시면 안될거에요! 대신 제가 관리하는 local DB 나 AWS RDB 의 kidskeeper_dev DB 의 경우 force:true 속성을 써서 DB 를 초기화시키거나, alter: true 속성으로 모델링 변경 사항만 반영하고는 있습니다.

지금 들어있는 데이터는 do migration으로 다시 삽입 가능해서 force방법을 고려중이었는데 alter:true가 더 효율적일 것 같네요! 그렇다면 app.js파일도 이 브랜치에서 바꿔놓는 것이 좋지 않을까요?현재는 force:true되어있습니다.

rxdcxdrnine commented 3 years ago

창구님 모델을 db에 적용시키는 것은 sync - force 쓰는 것 생각하고 계신가요?

현재 유빈님이 관리하시는 kidskeeper DB 에는 데이터가 담겨있으니 force: true 속성을 사용하시면 안될거에요!

대신 제가 관리하는 local DB 나 AWS RDB 의 kidskeeper_dev DB 의 경우 force:true 속성을 써서 DB 를 초기화시키거나, alter: true 속성으로 모델링 변경 사항만 반영하고는 있습니다.

지금 들어있는 데이터는 do migration으로 다시 삽입 가능해서 force방법을 고려중이었는데 alter:true가 더 효율적일 것 같네요! 그렇다면 app.js파일도 이 브랜치에서 바꿔놓는 것이 좋지 않을까요?현재는 force:true되어있습니다.

현재 브랜치에서 당장 필요한 코드는 아니지만, 유빈님이 develop 브랜치에 변경한 코드를 빠르게 반영하고 싶으신 경우엔 commit 해주시면 좋을 것 같습니다!

YubeenSon commented 3 years ago

창구님 모델을 db에 적용시키는 것은 sync - force 쓰는 것 생각하고 계신가요?

현재 유빈님이 관리하시는 kidskeeper DB 에는 데이터가 담겨있으니 force: true 속성을 사용하시면 안될거에요!

대신 제가 관리하는 local DB 나 AWS RDB 의 kidskeeper_dev DB 의 경우 force:true 속성을 써서 DB 를 초기화시키거나, alter: true 속성으로 모델링 변경 사항만 반영하고는 있습니다.

지금 들어있는 데이터는 do migration으로 다시 삽입 가능해서 force방법을 고려중이었는데 alter:true가 더 효율적일 것 같네요! 그렇다면 app.js파일도 이 브랜치에서 바꿔놓는 것이 좋지 않을까요?현재는 force:true되어있습니다.

현재 브랜치에서 당장 필요한 코드는 아니지만, 유빈님이 develop 브랜치에 변경한 코드를 빠르게 반영하고 싶으신 경우엔 commit 해주시면 좋을 것 같습니다!

당장 필요한 기능이 아니라면 지금 작업중인 브랜치에서 스키마 업데이트와 함께 변경사항 적용토록 하겠습니다!