feat(listen): Added user visible error messages for API failures

Closes #241. Affects update/save/delete operations for audio events and taggings
QutEcoacoustics · Oct 15, 2015 · ce1f435 · ce1f435
1 parent 824a3f4
commit ce1f435
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/src/app/listen/listen.js b/src/app/listen/listen.js
@@ -24,6 +24,7 @@ angular.module("bawApp.listen", ["decipher.tags", "ui.bootstrap.typeahead"])
         "UserProfileEvents",
         "Bookmark",
         "moment",
+        "growl",
         /**
          * The listen controller.
          * @param $scope
@@ -53,7 +54,7 @@ angular.module("bawApp.listen", ["decipher.tags", "ui.bootstrap.typeahead"])
             function ListenCtrl(
             $scope, $resource, $location, $routeParams, $route, $q, paths, constants, $url, _, ngAudioEvents,
             AudioRecording, MediaService, Media, AudioEvent, Tag, TagModel, Taggings, Site, Project, UserProfile,
-            UserProfileEvents, Bookmark, moment) {
+            UserProfileEvents, Bookmark, moment, growl) {
 
 
             const ChunkDurationSettings = constants.listen.chunkDurationSeconds;
@@ -632,6 +633,7 @@ angular.module("bawApp.listen", ["decipher.tags", "ui.bootstrap.typeahead"])
                         },
                         function error(response) {
                             console.error("Tagging creation failed", response);
+                            growl.error("Adding a tag to an annotation has failed. Please refresh the page. If you see this message often please let us know.");
                         });
                 });
                 $scope.$on("decipher.tags.addfailed", function (event) {
@@ -672,6 +674,7 @@ angular.module("bawApp.listen", ["decipher.tags", "ui.bootstrap.typeahead"])
                         },
                         function error(response) {
                             console.error("Tagging creation failed", response);
+                            growl.error("Removing a tag from an annotation has failed. Please refresh the page. If you see this message often please let us know.");
                         }
                     )
                     ;

diff --git a/src/components/directives/bawAnnotationViewer.js b/src/components/directives/bawAnnotationViewer.js
@@ -6,7 +6,8 @@ bawds.directive("bawAnnotationViewer",
         "AudioEvent",
         "Tag",
         "lodash",
-        function (paths, unitConverter, AudioEvent, Tag, _) {
+        "growl",
+        function (paths, unitConverter, AudioEvent, Tag, _, growl) {
 
             /**
              * Create an watcher for an audio event model.
@@ -261,6 +262,15 @@ bawds.directive("bawAnnotationViewer",
                             },
                             function error(response) {
                                 console.error("AnnotationEditor:modelUpdatesServer: " + action + " FAILURE", response);
+
+
+
+                                if (method === "update" && response.status === 404) {
+                                    growl.warning("The last annotation delete may not have worked. Try refreshing this page. If you see this message often please let us know.", {ttl: 5000});
+                                }
+                                else {
+                                    growl.error("The last annotation save/update has failed. Please refresh the page. If you see this message often please let us know.");
+                                }
                             });
                     };
                 };