better response handling (#323)

* better response handling

* fixed typo

* syntax fix

dist/leaflet.distortableimage.js

@@ -835,10 +835,14 @@ L.DistortableCollection = L.FeatureGroup.extend({
 
     // this may be overridden to update the UI to show export progress or completion
     function _defaultUpdater(data) {
+      data = JSON.parse(data);
       // optimization: fetch status directly from google storage:
-      if (data.hasOwnProperty('status_url') && statusUrl !== data.status_url && data.status_url.match('.json')) { statusUrl = data.status_url; }      if (data.status === "complete") {
+      if (statusUrl !== data.status_url && data.status_url.match('.json')) { statusUrl = data.status_url; }
+      if (data.status === "complete") {
         clearInterval(updateInterval);
-        alert("Export complete. " + data.jpg);
+      }
+      if (data.status === 'complete' && data.jpg !== null) {
+        alert("Export succeeded. http://export.mapknitter.org/" + data.jpg);
       }
       // TODO: update to clearInterval when status == "failed" if we update that in this file:
       // https://github.com/publiclab/mapknitter-exporter/blob/main/lib/mapknitterExporter.rb

package.json

@@ -1,6 +1,6 @@
 {
   "name": "leaflet-distortableimage",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "Leaflet plugin enabling image overlays to be distorted, stretched, and warped (built for Public Lab's MapKnitter: http://publiclab.org).",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"

src/DistortableCollection.js

@@ -353,10 +353,14 @@ L.DistortableCollection = L.FeatureGroup.extend({
 
     // this may be overridden to update the UI to show export progress or completion
     function _defaultUpdater(data) {
+      data = JSON.parse(data);
       // optimization: fetch status directly from google storage:
-      if (data.hasOwnProperty('status_url') && statusUrl !== data.status_url && data.status_url.match('.json')) { statusUrl = data.status_url; }      if (data.status === "complete") {
+      if (statusUrl !== data.status_url && data.status_url.match('.json')) { statusUrl = data.status_url; }
+      if (data.status === "complete") {
         clearInterval(updateInterval);
-        alert("Export complete. " + data.jpg);
+      }
+      if (data.status === 'complete' && data.jpg !== null) {
+        alert("Export succeeded. http://export.mapknitter.org/" + data.jpg);
       }
       // TODO: update to clearInterval when status == "failed" if we update that in this file:
       // https://github.com/publiclab/mapknitter-exporter/blob/main/lib/mapknitterExporter.rb