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

Fix: challenge_achievement member_id 컬럼 삭제 #101

Merged
merged 5 commits into from
Aug 23, 2024

Conversation

hee9841
Copy link
Collaborator

@hee9841 hee9841 commented Aug 23, 2024

🔗 이슈 연결

🚀 구현한 API

  • X

💡 반영할 내용 및 변경 사항 요약

  • challenge_achievement member_id 컬럼을 삭제 합니다.
  • challengeAchievement Entity에서 Member를 삭제합니다.
  • challengeAchievementRepository의 findByMemberIdAndRunningRecordId를 findByRunningRecordId로 수정하고 RunningRecordId로만 찾을 수 있게 변경합니다.

🔍 리뷰 요청/참고 사항

  • 챌린지 쪽은 완료 되어서 러닝 서비스 쪽에서 챌린지 작업 진행하시면 될 것 같아요.
  • 챌린지 쪽 테이블을 어떻게 구성할지 생각이 나서 구성 관련해서 새 이슈에 추가해 둘게요! 이슈 올라오면 해당 이슈에 리뷰 남겨주세요! 우선 러닝 쪽에서 서비스 호출하게 두고, 챌린지쪽 테이블 구성 확정되고 추 후에 리팩터링 작업들어가면 될 것 같아요.
  • 챌린지쪽 테이블 마이그레이션 작업과 리팩터링 작업은 마감까지 시간이 되면 진행하고 시간이 안될 것 같으면 배포 후 진행하는게 좋을 것 같은데 어떻게 생각하시나요?

@hee9841 hee9841 self-assigned this Aug 23, 2024
Copy link
Member

@Jaewon-pro Jaewon-pro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • 챌린지 쪽 테이블을 어떻게 구성할지 생각이 나서 구성 관련해서 새 이슈에 추가해 둘게요! 이슈 올라오면 해당 이슈에 리뷰 남겨주세요! 우선 러닝 쪽에서 서비스 호출하게 두고, 챌린지쪽 테이블 구성 확정되고 추 후에 리팩터링 작업들어가면 될 것 같아요.

넵 확인했습니다.

  • 리팩터링은 마감까지 시간이 되면 진행하고 시간이 안될 것 같으면 배포 후 진행하는게 좋을 것 같은데 어떻게 생각하시나요?

회원 탈퇴 리팩터링 말씀이시죠?
혼자뛰기, 배지 부여 기능이 개발된 뒤, 시간적 여유가 될 때 진행하면 좋을 것 같아요

Copy link
Member

@Jaewon-pro Jaewon-pro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

flyway sql에 5.0.4가 없어요, 5.0.5 대신 5.0.4를 써야 할 것 같아요

그리고 fk 조건 drop도 해야 해요

ALTER TABLE challenge_achievement
DROP CONSTRAINT fk_challenge_achievement_member,
    DROP COLUMN member_id;

Copy link
Member

@Jaewon-pro Jaewon-pro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 👍

@hee9841 hee9841 merged commit eb758ad into main Aug 23, 2024
1 check passed
@hee9841 hee9841 deleted the feature/#100/fix-challenge-achievement branch August 23, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants