adding logic for removing from localStorage after saving to s3 and db

relates to #163
pbywater · Aug 3, 2017 · 22dc096 · 22dc096
1 parent d82d023
commit 22dc096
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/public/scripts/s3-helpers.js b/public/scripts/s3-helpers.js
@@ -43,3 +43,8 @@ function dataURLtoFile(dataurl, filename) {
   }
   return new File([u8arr], filename, { type: mime });
 }
+
+function removeImagesFromStorage() {
+  localStorage.removeItem('imageToSave');
+  localStorage.removeItem('tagAndHeading');
+}
diff --git a/public/scripts/s3-uploads.js b/public/scripts/s3-uploads.js
@@ -29,6 +29,7 @@ function getSignedRequest(file) {
         imageUploadPending = true;
         updateTagAndHeading(response.imageId);
       } else {
+        removeImagesFromStorage();
         return new Error('Could not get signed URL.');
       }
     }
@@ -69,6 +70,7 @@ function updateTagAndHeading(imageId) {
   if (navigator.onLine && localStorage.getItem('imageTagAndHeading')) {
     const tagAndHeading = localStorage.getItem('imageTagAndHeading');
     addTagAndHeadingToDB(tagAndHeading.tag, tagAndHeading.heading, imageId);
+    localStorage.removeItem('imageToSave');
   }
 }
 
@@ -78,6 +80,10 @@ document.addEventListener('DOMContentLoaded', (event) => {
     if (file) {
       console.log('file is ', file);
       getSignedRequest(file);
+    } else {
+      localStorage.removeItem('imageTagAndHeading');
     }
+  } else if (localStorage.getItem('imageToSave')) {
+    localStorage.removeItem('imageToSave');
   }
 });