Merge branch 'lebihae/1228' into 'main'

fix(permissions): make documents access toggle actually trigger the mutation (fixes #1228) Closes #1228 See merge request churros/churros!276
inp-net · Nov 19, 2024 · 2b72804 · 2b72804
2 parents 5f47a42 + 11d9947
commit 2b72804
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/.changeset/bright-rocks-matter.md b/.changeset/bright-rocks-matter.md
@@ -0,0 +1,5 @@
+---
+'@churros/app': patch
+---
+
+fix(permissions): make documents access toggle actually trigger the mutation (fixes #1228)
diff --git a/packages/app/src/routes/(app)/users/[uid]/edit/permissions/+page.svelte b/packages/app/src/routes/(app)/users/[uid]/edit/permissions/+page.svelte
@@ -68,11 +68,10 @@
         <InputToggle
           slot="right"
           value={loading(user.canAccessDocuments, false)}
-          on:change={async ({ currentTarget }) => {
-            if (!(currentTarget instanceof HTMLInputElement)) return;
+          on:update={async ({ detail }) => {
             await mutateAndToast(SetCanAccessDocuments, {
               uid: $page.params.uid,
-              enabled: currentTarget.checked,
+              enabled: detail,
             });
           }}
         ></InputToggle>