Fixed the bug where todos will show in red if todo qa was created wit…

…h approved as null
b12io · Jul 27, 2018 · 903082a · 903082a
1 parent fbb5026
commit 903082a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/orchestra/static/dist/main.js b/orchestra/static/dist/main.js
@@ -61812,7 +61812,7 @@ function todoChecklist() {
       };
 
       scope.isInDanger = function (todo) {
-        return !todo.completed && _momentTimezone2.default.isBeforeNowBy(todo.due_datetime, 1, 'days') || scope.todoQas[todo.description] && !scope.todoQas[todo.description].approved;
+        return !todo.completed && _momentTimezone2.default.isBeforeNowBy(todo.due_datetime, 1, 'days') || scope.todoQas[todo.description] && scope.todoQas[todo.description].approved === false;
       };
 
       scope.isSkipped = function (todo) {

diff --git a/orchestra/static/orchestra/todos/todo-checklist.directive.es6.js b/orchestra/static/orchestra/todos/todo-checklist.directive.es6.js
@@ -35,7 +35,7 @@ export default function todoChecklist () {
       }
 
       scope.isInDanger = (todo) => {
-        return (!todo.completed && moment.isBeforeNowBy(todo.due_datetime, 1, 'days')) || (scope.todoQas[todo.description] && !scope.todoQas[todo.description].approved)
+        return (!todo.completed && moment.isBeforeNowBy(todo.due_datetime, 1, 'days')) || (scope.todoQas[todo.description] && scope.todoQas[todo.description].approved === false)
       }
 
       scope.isSkipped = (todo) => {