From 4fcc8a8a38dc7f0b3ce1bad86666617593fead95 Mon Sep 17 00:00:00 2001
From: Temnov Aleksey <temnov.aleksei@gmail.com>
Date: Sat, 13 Apr 2024 20:19:47 +0300
Subject: [PATCH] Add todos to overtime notification

---
 .../tapme/js/shared/models/overtime.notification.model.js     | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/examples/tapme/js/shared/models/overtime.notification.model.js b/examples/tapme/js/shared/models/overtime.notification.model.js
index 2957d94..c2644d7 100644
--- 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;