fix: make saved data track if the value has been changed

`opted_in` and `showExtensionInfo`, instead of storing a boolean value,
now store an object consisting of a `value` and `changed` keys. `value`
indicates which value the user wants for the given option, and `changed`
indicates whether the value has been changed from its default value.
Adding the `changed` key allows for users to have their options kept the
same if the default value is changed.

Signed-off-by: Lucas Sta Maria <[email protected]>

Author: priime0

src/js/helpers.js

@@ -223,9 +223,15 @@ async function saveGradesLocally (username, courses) {
     user_data["USERDATA_" + username] = { "courses": course_list };
 
     data.user_data = user_data;
-    data.opted_in = true;
+    data.opted_in = {
+        value: true,
+        changed: false,
+    };
+    data.showExtensionInfo = {
+        value: true,
+        changed: false,
+    };
     data.most_recent_user = username;
-    data.showExtensionInfo = true;
 
     browser.storage.local.set(data);
 }

src/js/saspowerschoolff.js

@@ -122,7 +122,7 @@ async function login_page () {
     browser.storage.local.get("showExtensionInfo").then(output => {
         new (Vue.extend(ExtensionInfo))({
             data: {
-                showInfo: output.showExtensionInfo,
+                showInfo: output.showExtensionInfo.value,
             },
         }).$mount('#saspes-info');
     });
@@ -132,7 +132,7 @@ async function login_page () {
     LastGradesDiv.id = "saspes-last-grades";
     document.body.appendChild(LastGradesDiv);
 
-    if ((await browser.storage.local.get("opted_in")).opted_in) {
+    if ((await browser.storage.local.get("opted_in")).opted_in.value) {
         (browser.storage.local.get("most_recent_user")).then(output => {
             const most_recent_user = output.most_recent_user;
             if (most_recent_user !== undefined) {