Add todos to overtime notification

temnov98 · Apr 13, 2024 · 4fcc8a8 · 4fcc8a8
1 parent c3a7bfb
commit 4fcc8a8
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/examples/tapme/js/shared/models/overtime.notification.model.js b/examples/tapme/js/shared/models/overtime.notification.model.js
@@ -1,3 +1,4 @@
+// todo: не нравится тут зависимость от trackerPageModel
 class OvertimeNotificationModel extends BaseModel {
     constructor() {
         super();
@@ -39,6 +40,9 @@ class OvertimeNotificationModel extends BaseModel {
         return (trackerPageModel.getTotalTimeInSeconds() * 1000) > this.overtimeTresholdInMilliseconds;
     }
 
+    // todo: плохо написал.
+    //  оно будет висеть 5 минут, если нет активной задачи, а должно обновляться мгновенно.
+    //  при исправлении важно учесть, чтобы не было циклических зависимостей
     _getShouldNotify() {
         if (!this.getIsOvertime()) {
             return false;