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;