Fix: challenge_achievement 테이블 분리, challenge enum에서 테이블로 변경 및 해당 테이블 변경으로 인한 수정

[hee9841]

🔗 이슈 연결

• 102

🚀 구현한 API

• X

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

• 챌린지 관련 테이블 변동 사항입니다.
  • challenge_achievement 의 퍼센티이지 정보를 challenge_achievement_percentage테이블로 분리합니다.
  • 오늘의 챌린지 데이터가 있는 challenge 테이블과 해당 챌린지에 관한 목표 조건 테이블인 challenge_goal_condition테이블을 생성합니다.
• 챌린지 조회 관련 변동 사항 입니다.(기존: enum에서 데이터를 가져옵니다.)
  • 챌린지의 종류만 보여주는 조회(challenge 만 조회 합니다.)
  • 저장 시 필요한 챌린지 조회(ChallengeData dto에 조회한 결과를 반환합니다.)
    • 저장 시 필요한 챌린지 정보는 challenge와 challenge_goal_condition(챌린지에 관한 목표 조건) 정보 입니다.
    • challenge와 challenge_goal_condition을 조인해서 가져옵니다.
    • challenge_goal_condition의 데이터는 arrayJson 형태로 조회해서 List<ChallengeCondition>로 변환합니다.

🔍 리뷰 요청/참고 사항

• 아래 코멘트 추가했습니다. 확인 부탁드립니다.

[Jaewon-pro]

이 PR에서 challenge 테이블 변경을 하기로 했는데, 필요하지 않은 변경 사항이 포함되어 있는 것 같아요

• 컨트롤러 계층에서 사용할 ChallengeAchievementResponse는 현재 사용하지 않아서, 지금 PR에서 같이 추가하지 않아도 될 것 같아요.
• 도메인 패키지에 있는 성취 관련 도메인 클래스들에 대해 dto 패키지를 새롭게 추가
  • 기존의 도메인 패키지에서는 dto 패키지 명을 사용하지 않았고, challenge/achievement/dto/ChallengeAchievement로 변경할 이유가 없어 보입니다.

[hee9841]

저희가 이슈에서 다룰 주제를 서로 잘못 이해고 있었네요😅

이번 pr은 일단 CRUD쪽도 같이 진행하는 것으로 할까요? 아니면 테이블 구성관련된 것만 바꾸고 CRUD관련된 PR은 따로 올릴까요??

pr올릴 때 작업 단위의 크기에 대해서도 정해두는게 좋을 것 같네요

[Jaewon-pro]

현재 ChallengeService에서 saveTest나 getChallengeAchievement는 컨트롤러나 테스트에서 사용되지 않아서 구현할 필요가 없을 것 같아요

우선 repository는 간단하게 구현해주시면 될 것 같아요 시간상 안될 것 같으면 TODO 주석으로 남겨주세요

[hee9841]

챌린지 성취 관련해서 저장, 조회 관련 repository는 이미 다 구현이 되어있고요. 어떤 함수인지 TODO 주석으로 남겨둘 게요! saveTest와 getChallengeAchievement 같은 경우 러닝 서비스 쪽에서 챌린지 성취 기록을 저장할 때 어떻게 저장하면 되는지 예시로 만들어놓은 함수에요! 주석으로 적어놓았습니다. 여기도 TODO 표시 해둘게요! 챌린지 성취 저장할 때 참고하시고 해당부분은 지우시면 됩니다!

그리고 지금 TODO 주석이랑 지금 pr리뷰 사항까지 해서 다시 커밋하겠습니다.

[hee9841]

러닝 저장 후 사용자 챌린지 성취 값 저장 로직은 ChallengeService의 saveTest()와 getChallengeAchievement()을 참고하시면 됩니다. 해당 메서드는 확인 후 삭제 부탁드립니다!

러닝 별 챌린지 성취 값 조회는 ChallengeAchievementRepository쪽 확인하시면 됩니다.

관련된 부분은 TODO로 주석 처리 및 설명 해두었습니다.