From 416d26658d11ed7886eb4582a96dd81e4a96e870 Mon Sep 17 00:00:00 2001
From: Henry Feng <henryfeng21@gmail.com>
Date: Thu, 17 Nov 2022 16:17:51 -0800
Subject: [PATCH 01/10] implementation of basic image local storage

---
 source/assets/scripts/main.js | 42 +++++++++++++++++++++++++++++++++++
 source/index.html             | 11 +++++----
 2 files changed, 49 insertions(+), 4 deletions(-)

diff --git a/source/assets/scripts/main.js b/source/assets/scripts/main.js
index 9d37c44..5fb44a4 100644
--- a/source/assets/scripts/main.js
+++ b/source/assets/scripts/main.js
@@ -43,6 +43,44 @@ function initFormHandler() {
 	//accessing form components
 	let tagContainer = document.getElementById("tag-container-form");
 	let form = document.querySelector("form");
+
+	/*
+	* change the input source of the image between local file and URL 
+	* depending on user's selection
+	*/
+	let select = document.getElementById("select");
+	select.addEventListener("change", function() {
+		const input = document.getElementById('source');
+	
+		if (select.value == "file") {
+		  input.innerHTML = `
+		  Source:
+		  <input type="file" id="mealImg" name="mealImg">
+		  `
+		}
+		else {
+		  input.innerHTML = `
+		  Source:
+		  <input type="text" id="mealImg" name="mealImg">
+		  `
+		}
+	});
+
+	//addressing sourcing image from local file
+	let imgDataURL = "";
+	document.getElementById("mealImg").addEventListener("change", function() {
+		const reader = new FileReader();
+		
+		//store image data URL after successful image load
+		reader.addEventListener("load", ()=>{
+			imgDataURL = reader.result;
+		}, false);
+		
+		//convert image file into data URL for local storage
+		reader.readAsDataURL(document.getElementById("mealImg").files[0]);
+	})
+
+
   
 	form.addEventListener("submit", function(){
 	/*
@@ -51,12 +89,16 @@ function initFormHandler() {
     */
 		let formData = new FormData(form);
 		let reviewObject = {};
+
 		for (let [key, value] of formData) {
 			console.log(`${key}`);
 			console.log(`${value}`);
 			if (`${key}` !== "tag-form") {
 				reviewObject[`${key}`] = `${value}`;
 			}
+			if (`${key}` === "mealImg" && select.value == "file") {
+				reviewObject["mealImg"] = imgDataURL;
+			}
 		}
 		reviewObject["tags"] = [];
 
diff --git a/source/index.html b/source/index.html
index 4418ea0..7708cb0 100644
--- a/source/index.html
+++ b/source/index.html
@@ -10,7 +10,6 @@
   <!-- Recipe Card Custom Element -->
   <script src="assets/scripts/ReviewCard.js" type="module"></script>
 
-
   <!-- Main Stylesheets & Scripts --> 
   <!-- Temporarily commented out reset.css due to furthur discussion needed on the values of the default config-->
   <!-- <link rel="stylesheet" href="/static/reset.css" /> -->
@@ -26,9 +25,14 @@
   <!----> <form id="new-food-entry">
     <fieldset>
       <legend>Pic:</legend>
-      <label for="mealImage">
+      Choose Input type:
+      <select id="select" name="select">
+        <option value="file">File Upload</option>
+        <option value="url">From an URL</option>
+      </select>
+      <label for="mealImage" id="source">
         Source:
-        <input type="text" id="mealImg" name="mealImg">
+        <input type="file" id="mealImg" name="mealImg">
       </label>
       <label for="image-alt">
         Alt Text:
@@ -36,7 +40,6 @@
       </label>
     </fieldset>
     <fieldset>
-
       <legend> Meal: </legend>
       <label for="Meal: ">Meal:
         <input type="text" id="mealName" name="mealName" required>

From f340f3ed8234c01bba0e3e40c4da7356b40f875b Mon Sep 17 00:00:00 2001
From: Henry Feng <henryfeng21@gmail.com>
Date: Sun, 20 Nov 2022 13:51:28 -0800
Subject: [PATCH 02/10] Added local img storage to updating

---
 source/ReviewDetails.html              |  9 ++++--
 source/assets/scripts/ReviewDetails.js | 42 ++++++++++++++++++++++++++
 2 files changed, 49 insertions(+), 2 deletions(-)

diff --git a/source/ReviewDetails.html b/source/ReviewDetails.html
index 8c457aa..609af99 100644
--- a/source/ReviewDetails.html
+++ b/source/ReviewDetails.html
@@ -26,9 +26,14 @@
     <!----> <form id="update-food-entry" class="hidden">
       <fieldset>
         <legend>Pic:</legend>
-        <label for="mealImage">
+        Choose Input type:
+        <select id="select" name="select">
+          <option value="file">File Upload</option>
+          <option value="url">From an URL</option>
+        </select>
+        <label for="mealImage" id="source">
           Source:
-          <input type="text" id="mealImg" name="mealImg">
+          <input type="file" accept="image/*" id="mealImg" name="mealImg">
         </label>
         <label for="image-alt">
           Alt Text:
diff --git a/source/assets/scripts/ReviewDetails.js b/source/assets/scripts/ReviewDetails.js
index 2f560f7..5a8f725 100644
--- a/source/assets/scripts/ReviewDetails.js
+++ b/source/assets/scripts/ReviewDetails.js
@@ -55,6 +55,45 @@ function setupUpdate(){
 				tagContainer.append(newTag);
 			}
 		}
+
+		/*
+		* change the input source of the image between local file and URL 
+		* depending on user's selection
+		*/
+		let select = document.getElementById("select");
+		select.addEventListener("change", function() {
+			const input = document.getElementById('source');
+		
+			if (select.value == "file") {
+			input.innerHTML = `
+			Source:
+			<input type="file" accept="image/*" id="mealImg" name="mealImg">
+			`
+			}
+			//TODO: change to photo taking for sprint 3
+			else {
+			input.innerHTML = `
+			Source:
+			<input type="text" id="mealImg" name="mealImg">
+			`
+			}
+		});
+
+		//addressing sourcing image from local file
+		let imgDataURL = "";
+		document.getElementById("mealImg").addEventListener("change", function() {
+			const reader = new FileReader();
+			
+			//store image data URL after successful image load
+			reader.addEventListener("load", ()=>{
+				imgDataURL = reader.result;
+			}, false);
+			
+			//convert image file into data URL for local storage
+			reader.readAsDataURL(document.getElementById("mealImg").files[0]);
+		})
+
+		
 		//Take formdata values as newData when submit
 		form.addEventListener("submit", function(){
 			/*
@@ -69,6 +108,9 @@ function setupUpdate(){
 				if (`${key}` !== "tag-form") {
 					newData[`${key}`] = `${value}`;
 				}
+				if (`${key}` === "mealImg" && select.value == "file") {
+					newData["mealImg"] = imgDataURL;
+				}
 			}
 			newData["tags"] = [];
 		

From 5d331c38000323b7dbb55ee2117d009d46438020 Mon Sep 17 00:00:00 2001
From: Marc <mreta0002@gmail.com>
Date: Sun, 20 Nov 2022 13:56:27 -0800
Subject: [PATCH 03/10] added for loops to tests

---
 source/assets/scripts/main.e2e.test.js | 411 +++++++++++++------------
 1 file changed, 217 insertions(+), 194 deletions(-)

diff --git a/source/assets/scripts/main.e2e.test.js b/source/assets/scripts/main.e2e.test.js
index ad177c9..339e93d 100644
--- a/source/assets/scripts/main.e2e.test.js
+++ b/source/assets/scripts/main.e2e.test.js
@@ -39,206 +39,229 @@ describe("test App end to end", async () => {
 
 	describe("test CRUD on simple inputs and default image", () => {
 
-		describe("test create 1 new review", async () => {
-			it("create 1 new review", async () => {
-				// Click the button to create a new review
-				let create_btn = await page.$("#create-btn");
-				await create_btn.click();
-				await page.waitForNavigation();
+		describe("test create 10 new reviews", async () => {
 
-				// create a new review
-				let review = {
-					imgAlt: "sample alt",
-					mealName: "sample name",
-					comments: "sample comment",
-					restaurant: "sample restaurant",
-					tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
-					rating: 1
-				}
-				await setReviewForm(page, review);
+			for (var i=0; i < 10; i++) {
 
-				// Click the save button to save updates
-				let save_btn = await page.$("#save-btn");
-				await save_btn.click();
-				await page.waitForNavigation();
-			});
+				it("create 1 new review", async () => {
 
-			it("check details page", async () => {
-				// check the details page for correctness
-				let expected = {
-					imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
-					imgAlt: "sample alt",
-					mealName: "sample name",
-					comments: "sample comment",
-					restaurant: "sample restaurant",
-					tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
-					rating: "http://localhost:8080/assets/images/icons/1-star.svg"
-				}
-				await checkCorrectness(page, "d", expected);
-			});
-		
-			it("check home page", async () => {
-				// Click the button to return to the home page
-				let home_btn = await page.$("#home-btn");
-				home_btn.click();
-				await page.waitForNavigation();
-
-				// Get the review card again and get its shadowRoot
-				let review_card = await page.$("review-card");
-				let shadowRoot = await review_card.getProperty("shadowRoot");
-
-				let expected = {
-					imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
-					imgAlt: "sample alt",
-					mealName: "sample name",
-					comments: "sample comment",
-					restaurant: "sample restaurant",
-					tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
-					rating: "http://localhost:8080/assets/images/icons/1-star.svg"
-				}
-				await checkCorrectness(shadowRoot, "a", expected);
-			});
-		});
-
-		describe("test read 1 review after refresh", async () => {
-			it("refresh page", async () => {
-				// Reload the page
-				await page.reload({ waitUntil: ["networkidle0", "domcontentloaded"] });
-			});
-
-			it("check details page", async () => {
-				// click review card
-				let review_card = await page.$("review-card");
-				await review_card.click();
-				await page.waitForNavigation();
-
-				// check the details page for correctness
-				let expected = {
-					imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
-					imgAlt: "sample alt",
-					mealName: "sample name",
-					comments: "sample comment",
-					restaurant: "sample restaurant",
-					tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
-					rating: "http://localhost:8080/assets/images/icons/1-star.svg"
-				}
-				await checkCorrectness(page, "d", expected);
-			});
-
-			it("check home page", async () => {
-				// Click the button to return to the home page
-				let home_btn = await page.$("#home-btn");
-				home_btn.click();
-				await page.waitForNavigation();
-
-				// Get the review card again and get its shadowRoot
-				let review_card = await page.$("review-card");
-				let shadowRoot = await review_card.getProperty("shadowRoot");
-
-				// check the details page for correctness
-				let expected = {
-					imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
-					imgAlt: "sample alt",
-					mealName: "sample name",
-					comments: "sample comment",
-					restaurant: "sample restaurant",
-					tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
-					rating: "http://localhost:8080/assets/images/icons/1-star.svg"
-				}
-				await checkCorrectness(shadowRoot, "a", expected);
-			});
-		});
-
-		describe("test update 1 review", async () => {
-
-			it("update 1 review", async () => {
-
-				// Get the only review card and click it
-				let review_card = await page.$("review-card");
-				await review_card.click();
-				await page.waitForNavigation();
-
-				// Click the button to show update form
-				let update_btn = await page.$("#update-btn");
-				await update_btn.click();
-
-				// create a new review
-				let review = {
-					imgAlt: "updated alt",
-					mealName: "updated name",
-					comments: "updated comment",
-					restaurant: "updated restaurant",
-					tags: ["tag -0", "tag -1", "tag -2", "tag -3", "tag -4", "tag -5"],
-					rating: 5
-				}
-				await setReviewForm(page, review);
-
-				// Click the save button to save updates
-				let save_btn = await page.$("#save-btn");
-				await save_btn.click();
-				await page.waitForNavigation();
-			});
-
-			it("check details page", async () => {
-				// check the details page for correctness
-				let expected = {
-					imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
-					imgAlt: "updated alt",
-					mealName: "updated name",
-					comments: "updated comment",
-					restaurant: "updated restaurant",
-					tags: ["tag -0", "tag -1", "tag -2", "tag -3", "tag -4", "tag -5"],
-					rating: "http://localhost:8080/assets/images/icons/5-star.svg"
-				}
-				await checkCorrectness(page, "d", expected);
-			});
-
-			it("check home page", async () => {
-				// Click the button to return to the home page
-				let home_btn = await page.$("#home-btn");
-				home_btn.click();
-				await page.waitForNavigation();
-
-				// Get the review card again and get its shadowRoot
-				let review_card = await page.$("review-card");
-				let shadowRoot = await review_card.getProperty("shadowRoot");
-
-				// check the details page for correctness
-				let expected = {
-					imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
-					imgAlt: "updated alt",
-					mealName: "updated name",
-					comments: "updated comment",
-					restaurant: "updated restaurant",
-					tags: ["tag -0", "tag -1", "tag -2", "tag -3", "tag -4", "tag -5"],
-					rating: "http://localhost:8080/assets/images/icons/5-star.svg"
-				}
-				await checkCorrectness(shadowRoot, "a", expected);
-			});
-
-		});
-
-		describe("test delete 1 review", async () => {
-			it("delete 1 review", async () => {
-				// Get the only review card and click it
-				let review_card = await page.$("review-card");
-				await review_card.click();
-				await page.waitForNavigation();
-
-				page.on('dialog', async dialog => {
-					console.log(dialog.message());
-					await dialog.accept();
+					// Click the button to create a new review
+					let create_btn = await page.$("#create-btn");
+					await create_btn.click();
+					await page.waitForNavigation();
+	
+					// create a new review
+					let review = {
+						imgAlt: "sample alt",
+						mealName: "sample name",
+						comments: "sample comment",
+						restaurant: "sample restaurant",
+						tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
+						rating: 1
+					}
+	
+					await setReviewForm(page, review);
+	
+					// Click the save button to save updates
+					let save_btn = await page.$("#save-btn");
+					await save_btn.click();
+					await page.waitForNavigation();
+				});
+	
+				it("check details page", async () => {
+					// check the details page for correctness
+					let expected = {
+						imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
+						imgAlt: "sample alt",
+						mealName: "sample name",
+						comments: "sample comment",
+						restaurant: "sample restaurant",
+						tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
+						rating: "http://localhost:8080/assets/images/icons/1-star.svg"
+					}
+					await checkCorrectness(page, "d", expected);
+				});
+			
+				it("check home page", async () => {
+					// Click the button to return to the home page
+					let home_btn = await page.$("#home-btn");
+					home_btn.click();
+					await page.waitForNavigation();
+	
+					// Get the review card again and get its shadowRoot
+					let review_card = await page.$("review-card");
+					let shadowRoot = await review_card.getProperty("shadowRoot");
+	
+					let expected = {
+						imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
+						imgAlt: "sample alt",
+						mealName: "sample name",
+						comments: "sample comment",
+						restaurant: "sample restaurant",
+						tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
+						rating: "http://localhost:8080/assets/images/icons/1-star.svg"
+					}
+					await checkCorrectness(shadowRoot, "a", expected);
 				});
 
-				// Get the delete button and click it
-				let delete_btn = await page.$("#delete-btn");
-				await delete_btn.click();
-				await page.waitForNavigation();
+			}
 
-				// Check that the card was correctly removed (there should be no remaining cards)
-				review_card = await page.$("#review-card");
-				assert.strictEqual(review_card, null);
-			});
+		});
+
+		describe("test read 10 reviews after refresh", async () => {
+
+			for (var i=0; i < 10; i++) {
+
+				it("refresh page", async () => {
+					// Reload the page
+					await page.reload({ waitUntil: ["networkidle0", "domcontentloaded"] });
+				});
+	
+				it("check details page", async () => {
+					// click review card
+					let review_card = await page.$("review-card");
+					await review_card.click();
+					await page.waitForNavigation();
+	
+					// check the details page for correctness
+					let expected = {
+						imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
+						imgAlt: "sample alt",
+						mealName: "sample name",
+						comments: "sample comment",
+						restaurant: "sample restaurant",
+						tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
+						rating: "http://localhost:8080/assets/images/icons/1-star.svg"
+					}
+					await checkCorrectness(page, "d", expected);
+				});
+	
+				it("check home page", async () => {
+					// Click the button to return to the home page
+					let home_btn = await page.$("#home-btn");
+					home_btn.click();
+					await page.waitForNavigation();
+	
+					// Get the review card again and get its shadowRoot
+					let review_card = await page.$("review-card");
+					let shadowRoot = await review_card.getProperty("shadowRoot");
+	
+					// check the details page for correctness
+					let expected = {
+						imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
+						imgAlt: "sample alt",
+						mealName: "sample name",
+						comments: "sample comment",
+						restaurant: "sample restaurant",
+						tags: ["tag 0", "tag 1", "tag 2", "tag 3", "tag 4"],
+						rating: "http://localhost:8080/assets/images/icons/1-star.svg"
+					}
+					await checkCorrectness(shadowRoot, "a", expected);
+				});
+
+			}
+
+		});
+
+		describe("test update 10 reviews", async () => {
+
+			for (var i=0; i < 10; i++) {
+
+				it("update 1 review", async () => {
+
+					// Get the only review card and click it
+					let review_card = await page.$("review-card");
+					await review_card.click();
+					await page.waitForNavigation();
+	
+					// Click the button to show update form
+					let update_btn = await page.$("#update-btn");
+					await update_btn.click();
+	
+					// create a new review
+					let review = {
+						imgAlt: "updated alt",
+						mealName: "updated name",
+						comments: "updated comment",
+						restaurant: "updated restaurant",
+						tags: ["tag -0", "tag -1", "tag -2", "tag -3", "tag -4", "tag -5"],
+						rating: 5
+					}
+					await setReviewForm(page, review);
+	
+					// Click the save button to save updates
+					let save_btn = await page.$("#save-btn");
+					await save_btn.click();
+					await page.waitForNavigation();
+				});
+	
+				it("check details page", async () => {
+					// check the details page for correctness
+					let expected = {
+						imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
+						imgAlt: "updated alt",
+						mealName: "updated name",
+						comments: "updated comment",
+						restaurant: "updated restaurant",
+						tags: ["tag -0", "tag -1", "tag -2", "tag -3", "tag -4", "tag -5"],
+						rating: "http://localhost:8080/assets/images/icons/5-star.svg"
+					}
+					await checkCorrectness(page, "d", expected);
+				});
+	
+				it("check home page", async () => {
+					// Click the button to return to the home page
+					let home_btn = await page.$("#home-btn");
+					home_btn.click();
+					await page.waitForNavigation();
+	
+					// Get the review card again and get its shadowRoot
+					let review_card = await page.$("review-card");
+					let shadowRoot = await review_card.getProperty("shadowRoot");
+	
+					// check the details page for correctness
+					let expected = {
+						imgSrc: "http://localhost:8080/assets/images/icons/plate_with_cutlery.png",
+						imgAlt: "updated alt",
+						mealName: "updated name",
+						comments: "updated comment",
+						restaurant: "updated restaurant",
+						tags: ["tag -0", "tag -1", "tag -2", "tag -3", "tag -4", "tag -5"],
+						rating: "http://localhost:8080/assets/images/icons/5-star.svg"
+					}
+					await checkCorrectness(shadowRoot, "a", expected);
+				});
+
+			}			
+
+		});
+
+		describe("test delete 10 reviews", async () => {
+
+			for (var i=0; i < 10; i++) {
+
+				it("delete 1 review", async () => {
+					// Get the only review card and click it
+					let review_card = await page.$("review-card");
+					await review_card.click();
+					await page.waitForNavigation();
+	
+					page.on('dialog', async dialog => {
+						console.log(dialog.message());
+						await dialog.accept();
+					});
+	
+					// Get the delete button and click it
+					let delete_btn = await page.$("#delete-btn");
+					await delete_btn.click();
+					await page.waitForNavigation();
+	
+					// Check that the card was correctly removed (there should be no remaining cards)
+					review_card = await page.$("#review-card");
+					assert.strictEqual(review_card, null);
+				});
+
+			}
 		});
 
 	});

From f2edc4dc6a7a97d367b0349a5a6e005f2118cc8f Mon Sep 17 00:00:00 2001
From: Marc <mreta0002@gmail.com>
Date: Sun, 20 Nov 2022 14:01:48 -0800
Subject: [PATCH 04/10] added for loops for multiple tests

---
 source/assets/scripts/main.e2e.test.js | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/source/assets/scripts/main.e2e.test.js b/source/assets/scripts/main.e2e.test.js
index 339e93d..85ac54f 100644
--- a/source/assets/scripts/main.e2e.test.js
+++ b/source/assets/scripts/main.e2e.test.js
@@ -40,11 +40,8 @@ describe("test App end to end", async () => {
 	describe("test CRUD on simple inputs and default image", () => {
 
 		describe("test create 10 new reviews", async () => {
-
 			for (var i=0; i < 10; i++) {
-
 				it("create 1 new review", async () => {
-
 					// Click the button to create a new review
 					let create_btn = await page.$("#create-btn");
 					await create_btn.click();
@@ -103,15 +100,11 @@ describe("test App end to end", async () => {
 					}
 					await checkCorrectness(shadowRoot, "a", expected);
 				});
-
 			}
-
 		});
 
 		describe("test read 10 reviews after refresh", async () => {
-
 			for (var i=0; i < 10; i++) {
-
 				it("refresh page", async () => {
 					// Reload the page
 					await page.reload({ waitUntil: ["networkidle0", "domcontentloaded"] });
@@ -158,17 +151,12 @@ describe("test App end to end", async () => {
 					}
 					await checkCorrectness(shadowRoot, "a", expected);
 				});
-
 			}
-
 		});
 
 		describe("test update 10 reviews", async () => {
-
 			for (var i=0; i < 10; i++) {
-
 				it("update 1 review", async () => {
-
 					// Get the only review card and click it
 					let review_card = await page.$("review-card");
 					await review_card.click();
@@ -231,15 +219,11 @@ describe("test App end to end", async () => {
 					}
 					await checkCorrectness(shadowRoot, "a", expected);
 				});
-
 			}			
-
 		});
 
 		describe("test delete 10 reviews", async () => {
-
 			for (var i=0; i < 10; i++) {
-
 				it("delete 1 review", async () => {
 					// Get the only review card and click it
 					let review_card = await page.$("review-card");
@@ -260,10 +244,8 @@ describe("test App end to end", async () => {
 					review_card = await page.$("#review-card");
 					assert.strictEqual(review_card, null);
 				});
-
 			}
 		});
-
 	});
 
 	after(async () => {

From d284f0ddf231a468edcb59009508ccdafb5bd82a Mon Sep 17 00:00:00 2001
From: Arthur Lu <learthurgo@gmail.com>
Date: Sun, 20 Nov 2022 14:17:10 -0800
Subject: [PATCH 05/10] change some npm scripts

Signed-off-by: Arthur Lu <learthurgo@gmail.com>
---
 package.json | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/package.json b/package.json
index 6ea0de5..be9a691 100644
--- a/package.json
+++ b/package.json
@@ -4,10 +4,11 @@
   "type": "module",
   "scripts": {
     "test": "mocha --recursive --require mock-local-storage './{,!(node_modules)/**}/*.test.js'",
-    "lint": "eslint **/*.js",
-    "fix-style": "eslint --fix **/*.js",
-    "lintHTML": "htmlhint **/*.html",
-    "lintCSS": "stylelint **/*.css",
+    "lint-js": "eslint **/*.js",
+    "fix-js": "eslint --fix **/*.js",
+    "lint-html": "htmlhint **/*.html",
+    "lint-css": "stylelint **/*.css",
+    "fix-css": "stylelint --fix **/*.css",
     "http-server": "http-server source"
   },
   "devDependencies": {

From 3a7ed2fb1671c6075b2227c44829de2624a595ca Mon Sep 17 00:00:00 2001
From: Marc <mreta0002@gmail.com>
Date: Sun, 20 Nov 2022 14:26:48 -0800
Subject: [PATCH 06/10] fix stylelint to allow tabs as indent

---
 .stylelintrc.json | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/.stylelintrc.json b/.stylelintrc.json
index 2f1eeda..76eba38 100644
--- a/.stylelintrc.json
+++ b/.stylelintrc.json
@@ -1,3 +1,4 @@
 {
-	"extends": "stylelint-config-standard"
+	"extends": "stylelint-config-standard",
+	"ignore": ["inside-parens", "param", "value"]
 }
\ No newline at end of file

From 3c7b5f7f4ee66c88312fd0d60103e9cb6a62cb24 Mon Sep 17 00:00:00 2001
From: Henry Feng <henryfeng21@gmail.com>
Date: Sun, 20 Nov 2022 14:27:24 -0800
Subject: [PATCH 07/10] addresses case where image is not updated

---
 source/assets/scripts/ReviewDetails.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/source/assets/scripts/ReviewDetails.js b/source/assets/scripts/ReviewDetails.js
index 5a8f725..60cb614 100644
--- a/source/assets/scripts/ReviewDetails.js
+++ b/source/assets/scripts/ReviewDetails.js
@@ -82,6 +82,7 @@ function setupUpdate(){
 		//addressing sourcing image from local file
 		let imgDataURL = "";
 		document.getElementById("mealImg").addEventListener("change", function() {
+			console.log("reading used");
 			const reader = new FileReader();
 			
 			//store image data URL after successful image load
@@ -108,7 +109,11 @@ function setupUpdate(){
 				if (`${key}` !== "tag-form") {
 					newData[`${key}`] = `${value}`;
 				}
-				if (`${key}` === "mealImg" && select.value == "file") {
+				//Account for the case where image is not updated
+				if (`${key}` === "mealImg" && document.getElementById("mealImg").value === "") {
+					newData["mealImg"] = currReview["mealImg"];
+				}
+				else if (`${key}` === "mealImg" && select.value == "file") {
 					newData["mealImg"] = imgDataURL;
 				}
 			}

From c5cc7bf30984b0cf320d3ff5ea1caba6e9ba843b Mon Sep 17 00:00:00 2001
From: Arthur Lu <learthurgo@gmail.com>
Date: Sun, 20 Nov 2022 14:36:13 -0800
Subject: [PATCH 08/10] update github actions with new npm script naming

Signed-off-by: Arthur Lu <learthurgo@gmail.com>
---
 .github/workflows/css-linting.yml  | 2 +-
 .github/workflows/html-linting.yml | 2 +-
 .github/workflows/js-linting.yml   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/css-linting.yml b/.github/workflows/css-linting.yml
index fae3b66..48b333a 100644
--- a/.github/workflows/css-linting.yml
+++ b/.github/workflows/css-linting.yml
@@ -20,4 +20,4 @@ jobs:
       - name: Install dependencies
         run: sudo npm install
       - name: Run tests
-        run: sudo npm run lintCSS
\ No newline at end of file
+        run: sudo npm run lint-css
\ No newline at end of file
diff --git a/.github/workflows/html-linting.yml b/.github/workflows/html-linting.yml
index ab68590..c24292b 100644
--- a/.github/workflows/html-linting.yml
+++ b/.github/workflows/html-linting.yml
@@ -20,4 +20,4 @@ jobs:
       - name: Install dependencies
         run: sudo npm install
       - name: Run tests
-        run: sudo npm run lintHTML
+        run: sudo npm run lint-html
diff --git a/.github/workflows/js-linting.yml b/.github/workflows/js-linting.yml
index f3aafaf..b642926 100644
--- a/.github/workflows/js-linting.yml
+++ b/.github/workflows/js-linting.yml
@@ -22,4 +22,4 @@ jobs:
       - name: Install dependencies
         run: sudo npm install
       - name: Run tests
-        run: sudo npm run lint
\ No newline at end of file
+        run: sudo npm run lint-js
\ No newline at end of file

From 12e818e079dfa578e74b5b922edf2285547d4bb4 Mon Sep 17 00:00:00 2001
From: Arthur Lu <learthurgo@gmail.com>
Date: Sun, 20 Nov 2022 15:23:22 -0800
Subject: [PATCH 09/10] fix js linting

Signed-off-by: Arthur Lu <learthurgo@gmail.com>
---
 source/assets/scripts/CreatePage.js    | 20 ++++++++++----------
 source/assets/scripts/ReviewDetails.js | 20 ++++++++++----------
 2 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/source/assets/scripts/CreatePage.js b/source/assets/scripts/CreatePage.js
index 02ddea1..9f5bb65 100644
--- a/source/assets/scripts/CreatePage.js
+++ b/source/assets/scripts/CreatePage.js
@@ -22,20 +22,20 @@ function initFormHandler() {
 	*/
 	let select = document.getElementById("select");
 	select.addEventListener("change", function() {
-		const input = document.getElementById('source');
+		const input = document.getElementById("source");
 	
 		if (select.value == "file") {
-		  input.innerHTML = `
-		  Source:
-		  <input type="file" accept="image/*" id="mealImg" name="mealImg">
-		  `
+			input.innerHTML = `
+			Source:
+			<input type="file" accept="image/*" id="mealImg" name="mealImg">
+			`;
 		}
 		//TODO: change to photo taking for sprint 3
 		else {
-		  input.innerHTML = `
-		  Source:
-		  <input type="text" id="mealImg" name="mealImg">
-		  `
+			input.innerHTML = `
+			Source:
+			<input type="text" id="mealImg" name="mealImg">
+			`;
 		}
 	});
 
@@ -51,7 +51,7 @@ function initFormHandler() {
 		
 		//convert image file into data URL for local storage
 		reader.readAsDataURL(document.getElementById("mealImg").files[0]);
-	})
+	});
 		
 	form.addEventListener("submit", function(e){
 	/*
diff --git a/source/assets/scripts/ReviewDetails.js b/source/assets/scripts/ReviewDetails.js
index 60cb614..1786c5e 100644
--- a/source/assets/scripts/ReviewDetails.js
+++ b/source/assets/scripts/ReviewDetails.js
@@ -62,20 +62,20 @@ function setupUpdate(){
 		*/
 		let select = document.getElementById("select");
 		select.addEventListener("change", function() {
-			const input = document.getElementById('source');
+			const input = document.getElementById("source");
 		
 			if (select.value == "file") {
-			input.innerHTML = `
-			Source:
-			<input type="file" accept="image/*" id="mealImg" name="mealImg">
-			`
+				input.innerHTML = `
+				Source:
+				<input type="file" accept="image/*" id="mealImg" name="mealImg">
+				`;
 			}
 			//TODO: change to photo taking for sprint 3
 			else {
-			input.innerHTML = `
-			Source:
-			<input type="text" id="mealImg" name="mealImg">
-			`
+				input.innerHTML = `
+				Source:
+				<input type="text" id="mealImg" name="mealImg">
+				`;
 			}
 		});
 
@@ -92,7 +92,7 @@ function setupUpdate(){
 			
 			//convert image file into data URL for local storage
 			reader.readAsDataURL(document.getElementById("mealImg").files[0]);
-		})
+		});
 
 		
 		//Take formdata values as newData when submit

From 7b7abad2e6fd6a554171414f4737d2967905c201 Mon Sep 17 00:00:00 2001
From: Arthur Lu <learthurgo@gmail.com>
Date: Sun, 20 Nov 2022 15:33:31 -0800
Subject: [PATCH 10/10] fix error in test syntax

Signed-off-by: Arthur Lu <learthurgo@gmail.com>
---
 source/assets/scripts/main.e2e.test.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/source/assets/scripts/main.e2e.test.js b/source/assets/scripts/main.e2e.test.js
index 004f1b7..594d359 100644
--- a/source/assets/scripts/main.e2e.test.js
+++ b/source/assets/scripts/main.e2e.test.js
@@ -229,7 +229,7 @@ describe("test App end to end", async () => {
 					console.log(dialog.message());
 					await dialog.accept();
 				});
-			}
+			});
 		});
 	});