Wake-up-together-TogetUp / togetup-server

AI를 활용한 커뮤니티 기반 미션 알람 서비스, TogetUp!
1 stars 1 forks source link

[FEATURE] 비활성화된 알람도 포함하는 알람 타임라인 조회 기능 구현 #141

Closed 05AM closed 3 months ago

05AM commented 4 months ago

☀️ 작업 사항

이전 타임라인에서 누락되었던 비활성화된 알람도 포함하는 타임라인 조회 기능을 구현했습니다.

☀️ 관련 이슈

resolved : #79

☀️ 참고 사항

알람 종류

  1. 활성 요일 알람
  2. 오늘 울리고 비활성화된 요일 알람
  3. 일회성 알람
  4. 오늘 울리고 비활성화된 일회성 알람

과정

이전 회의에서 일회성 알람 중 울리고 비활성화된 알람은 오늘의 타임라인에 포함되지 않는 버그에 대해서 해결방법을 논의했는데요, deactivated time을 추가하는 방식으로 결론이 났었습니다.

하지만 여러 경우의 수를 떠올리다보니 사용자가 알람을 의미없이 여러 번 활성화/비활성화 하다보면 해당 방법이 논리적으로 무결하지 않다는 결론에 이르게 되었습니다.

예를들어 울릴 시간이 지난 비활성된 일회성 알람의 경우 사용자가 이후 활성화 후 비활성화 한다면 이것도 오늘 울린 알람에 포함이 되게 됩니다. 일회성 알람뿐만 아니라 요일별 알람도 비슷한 문제가 있습니다.

결론

그래서 알람 타임라인이라는 기능에 초점을 맞춰 다시 생각해봤을 때 현재 시간 이전의 알람은 알람을 수행하여 기록이 남은 경우만 타임라인에 표시되고, 이후의 알람은 이전과 동일하게 오늘 예정된 활성화된 알람을 표시하는 것이 구현가능한 정책 중 가장 바람직하다고 생각이 들었습니다.

물론 오늘 울렸지만 수행하지 않은 알람까지 표시된다면 가장 좋다고 생각하지만, 해당 방법은 알람이 울린 시간을 남기지 않는 방법 이외에는 무결한 해결 방법이 떠오르지 않았습니다. 혹시 좋은 아이디어가 있다면 추천해주시면 좋을 것 같습니다.

하지만 해당 알람은 그렇게 큰 의미를 가지고 있지는 않은 것 같습니다. 또한 의미가 없다고 생각하는 것만큼 해당 기능에만 사용되는 새로운 컬럼이나 테이블을 추가해서 관리하는 것도 불필요하다는 의견입니다.

우선 구현은 이렇게 했지만 추가 의견이 있다면 논의하고 싶습니다!

또한 오늘 정기 회의에서 다른 팀원들과 해당 방법에 대해서 또 다시 논의해보면 좋을 것 같습니다!

hye-on commented 4 months ago

또한 오늘 정기 회의에서 다른 팀원들과 해당 방법에 대해서 또 다시 논의해보면 좋을 것 같습니다!

고생하셨습니다. 고민 많이 하셨을 것 같아요 👍 네 오늘 회의에서 이야기보면 좋을 것 같아요!