[ADD] 데일리 루틴 달성 취소 로직 추가

[thguss]

✨ Related Issue

• close [FIX] 데일리 루틴 달성 기능 수정 #274
  
  

📝 기능 구현 명세

🐥 추가적인 언급 사항

• 캐시 누락 사유
  • 캐시를 사용하기 위해서는 복구를 위한 원본 데이터가 필요합니다.
  • 의논한 로직에서는 원본 데이터(ex. 취소 이력)를 담고 있지 않기 때문에 캐싱 장애 발생 시 데이터를 복구할 길이 없습니다.
  • 따라서 캐싱을 진행하지 않고, isAchieveToday 칼럼 하나로 로직 체크가 충분히 가능하다고 판단하여 이처럼 진행했습니다.

[Chan531]

수고하셨습니다! 혹시 캐시 미사용 이유에 대한 설명을 pr에 담아주신다했는데 누락된걸까요?!

[Chan531]

수고하셨습니다! 승인해놓겠습니다!

마지막으로 궁금한 점이 두 가지 있는데 답변 주시면 감사하겠습니다!

1. achieveMemberRoutine이 MemberRoutine의 isAchieve의 상태를 변경하는 메소드로 바뀌었는데 기존 이름을 유지해도 괜찮을지
2. deleteMemberRoutineIfTypeIsOneTime이 MemberRoutine의 RoutineType이 CHALLENGE인지를 확인하는 역할을 하던데 왜 deleteMemberRoutineIfTypeIsChallenge가 아닌지 (혹시 Maker를 염두하고 작성하신건지)

[thguss]

1. 동의합니다. updateAchievementStatus는 어떨까요?
2. 일회성 달성(달성하면 삭제)에 해당하는 루틴은 삭제하는 로직으로 작성한 메서드입니다. 현재는 CHALLENGE 타입일 경우에만 일회성 달성이 적용되어 CHALLENGE인 경우만 체크되어 있습니다. 재활용성을 위해 CHALLENGE 대신 일회성으로 적용했는데 다른 의견 있으신지 궁금합니다!

[Chan531]

@thguss

1. 좋아요!
2. 뭔가 CHALLENGE만이 일회성일 거 같아서 메소드명을 바꿔도 괜찮을 거 같고 재활용성을 염두하신거라면 그냥 둬도 괜찮을 거 같습니다! 편하신대로 해주시면 좋을 거 같아요!