From 9cc30f25e4bf34287f930c69d52eb91145eec4d2 Mon Sep 17 00:00:00 2001
From: Graham Steffaniak <42989099+gtsteffaniak@users.noreply.github.com>
Date: Mon, 6 Jan 2025 15:57:31 -0500
Subject: [PATCH] fix user creation condition (#275)

---
 frontend/src/components/settings/UserForm.vue | 20 +++++++++----------
 frontend/src/views/settings/User.vue          |  6 +++---
 makefile                                      |  2 +-
 3 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/frontend/src/components/settings/UserForm.vue b/frontend/src/components/settings/UserForm.vue
index 0dfbbd30..1ad843c5 100644
--- a/frontend/src/components/settings/UserForm.vue
+++ b/frontend/src/components/settings/UserForm.vue
@@ -1,14 +1,14 @@
 <template>
-  <div v-if="!user.perm.admin">
+  <div v-if="!user.perm.admin && !isNew">
     <label for="password">{{ $t("settings.password") }}</label>
-      <input
-        class="input input--block"
-        type="password"
-        placeholder="enter new password"
-        v-model="user.password"
-        id="password"
-        @input="emitUpdate"
-      />
+    <input
+      class="input input--block"
+      type="password"
+      placeholder="enter new password"
+      v-model="user.password"
+      id="password"
+      @input="emitUpdate"
+    />
   </div>
   <div v-else>
     <p v-if="!isDefault">
@@ -104,7 +104,7 @@ export default {
   watch: {
     user: {
       handler(newUser) {
-        this.localUser = { ...newUser };  // Watch for changes in the parent and update the local copy
+        this.localUser = { ...newUser }; // Watch for changes in the parent and update the local copy
       },
       immediate: true,
       deep: true,
diff --git a/frontend/src/views/settings/User.vue b/frontend/src/views/settings/User.vue
index 129f3bfd..d251b7df 100644
--- a/frontend/src/views/settings/User.vue
+++ b/frontend/src/views/settings/User.vue
@@ -111,14 +111,14 @@ export default {
       try {
         if (this.isNew) {
           const loc = await usersApi.create(this.userPayload); // Use the computed property
-          this.$router.push({ path: loc });
+          this.$router.push({ path: "/settings", hash: "#users-main" });
           notify.showSuccess(this.$t("settings.userCreated"));
         } else {
           let which = ["all"];
           if (!this.user.perm.admin) {
-            which = ["password"]
+            which = ["password"];
           }
-          await usersApi.update(this.userPayload,which);
+          await usersApi.update(this.userPayload, which);
           notify.showSuccess(this.$t("settings.userUpdated"));
         }
       } catch (e) {
diff --git a/makefile b/makefile
index e7b69c7b..0f23fd71 100644
--- a/makefile
+++ b/makefile
@@ -2,7 +2,7 @@
 setup:
 	echo "creating ./backend/test_config.yaml for local testing..." && \
 	if [ ! -f backend/test__config.yaml ]; then \
-		cp backend/filebrowser.yaml backend/test_config.yaml; \
+		cp backend/config.yaml backend/test_config.yaml; \
 	fi
 	echo "installing swagger needed to generate backend api docs..." && \
 	go install github.com/swaggo/swag/cmd/swag@latest && \