-
Notifications
You must be signed in to change notification settings - Fork 0
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
[REFACTOR] 기획 의도에 맞게 erd 수정 #104
Conversation
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.
확인했습니다~ approve해도 될 것 같아요.
records 에 score 가 아직 살아있는데 해당 부분을 어떻게 처리해야 할지 고민이 됩니당
그냥 필요없으니 drop하면 될 것 같습니다. lineup_player_id
도 같이요.
운영 DB에 Record 데이터가 남아 있었다면 함부로 drop하면 안되지만 확인해보니 데이터가 없었어요. 그냥 drop하면 될 것 같습니다.
테스트가 깨지는건 마찬가지로 잠시 disabled 해두시면 될 것 같습니다. 타임라인 조회 api를 수정하고 그 기능을 수정하는 사람이 다시 disabled를 없애고 테스트를 통과시키도록 바꾸면 좋을 것 같아요.
아니면 api 수정할 때 함께 drop하도록 해도 될 것 같네요.
🌍 이슈 번호
📝 구현 내용
🍀 확인해야 할 부분