Skip to content
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

Merged
merged 18 commits into from
Feb 10, 2024
Merged

Conversation

Jin409
Copy link
Contributor

@Jin409 Jin409 commented Feb 8, 2024

🌍 이슈 번호

📝 구현 내용

  • lineup_players 에 number, is_captain 추가
  • league_team_players 에 number 추가
  • record 에 record_type 추가
  • record 에 score 관련 모두 recorded 로 변경
  • score_records 테이블 추가
  • replacement_records 테이블 추가

🍀 확인해야 할 부분

  • records 에 score 가 아직 살아있는데 해당 부분을 어떻게 처리해야 할지 고민이 됩니당
  • 아직 league 나 game 에서 경기 결과 반영하는 부분은 수정하지 않았습니답. 기획 수정쪽으로 얘기 마치면 수정하는 게 어떨까 해요

@Jin409 Jin409 changed the title Refactor/#103 erd [REFACTOR] 기획 의도에 맞게 erd 수정 Feb 8, 2024
@Jin409 Jin409 marked this pull request as ready for review February 8, 2024 20:57
@Jin409 Jin409 requested a review from ldk980130 February 8, 2024 20:57
@Jin409 Jin409 self-assigned this Feb 8, 2024
Copy link
Contributor

@ldk980130 ldk980130 left a 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하도록 해도 될 것 같네요.

@Jin409 Jin409 merged commit bcdf24b into main Feb 10, 2024
1 check passed
@Jin409 Jin409 deleted the refactor/#103-erd branch February 10, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 기획 의도에 맞게 erd 수정
2 participants