fix settings saving

omnibear · aciccarello · Nov 16, 2022 · Nov 17, 2022 · Nov 18, 2022 · Nov 24, 2024
commit 934d15256819137fae515965345687a7f2f5caa3
diff --git a/src/components/settings/EmojiSettings.jsx b/src/components/settings/EmojiSettings.jsx
@@ -6,17 +6,15 @@ export default function EmojiSettings() {
   const settings = useContext(Settings);
 
   const deleteReacji = (index) => {
-    const reacji = settings.reacji;
+    const reacji = settings.reacji.value;
     const before = reacji.slice(0, index);
     const after = reacji.slice(index + 1);
-    settings.setReacji(before.concat(after));
-    settings.save();
+    settings.reacji.value = before.concat(after);
   };
 
   const addReacji = () => {
-    if (value && settings.reacji.indexOf(value) === -1) {
+    if (value && !settings?.reacji.value?.includes(value)) {
       settings.addReacji(value);
-      settings.save();
       setValue("");
     }
   };

diff --git a/src/components/settings/EndpointFields.jsx b/src/components/settings/EndpointFields.jsx
@@ -1,5 +1,5 @@
-import { useState, useContext } from 'preact/hooks';
-import Settings from '../../contexts/Settings';
+import { useState, useContext } from "preact/hooks";
+import Settings from "../../contexts/Settings";
 
 export default function EndpointFields() {
   const [showFields, setShowFields] = useState(false);
@@ -21,7 +21,9 @@ export default function EndpointFields() {
                 id="slug"
                 type="text"
                 value={settings.slugFieldName}
-                onChange={(e) => settings.setSlugFieldName(e.target.value)}
+                onChange={(e) =>
+                  (settings.slugFieldName.value = e.target.value)
+                }
               />
               <div className="settings-form__description">
                 Choose the name of the field that the slug will be sent in. This
@@ -35,7 +37,7 @@ export default function EndpointFields() {
                 type="text"
                 value={settings.syndicateToFieldName}
                 onChange={(e) =>
-                  settings.setSyndicateToFieldName(e.target.value)
+                  (settings.syndicateToFieldName.value = e.target.value)
                 }
               />
               <div className="settings-form__description">
@@ -48,7 +50,7 @@ export default function EndpointFields() {
         : null}
       <div className="text-right">
         <button type="button" onClick={() => setShowFields(!showFields)}>
-          {showFields ? 'Hide' : 'Show'}
+          {showFields ? "Hide" : "Show"}
         </button>
       </div>
     </fieldset>