Fix photo update UI/UX bug

ewhanson · Mar 15, 2023 · 653b811 · 653b811
1 parent daf45ca
commit 653b811
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 3 deletions.
diff --git a/ui/src/components/auth/UploaderAuth.jsx b/ui/src/components/auth/UploaderAuth.jsx
@@ -18,7 +18,7 @@ export function UploaderAuth() {
 
   const onSubmit = async (event) => {
     event.preventDefault();
-    setIsSubmitting(false);
+    setIsSubmitting(true);
     setErrorMessage("");
 
     try {

diff --git a/ui/src/components/routes/UploaderLogin.jsx b/ui/src/components/routes/UploaderLogin.jsx
@@ -1,7 +1,23 @@
 import { MainComponentWrapper } from "../MainComponentWrapper.jsx";
 import { UploaderAuth } from "../auth/UploaderAuth.jsx";
+import { useUploaderAuthProtected } from "../../lib/customHooks.js";
+import { route } from "preact-router";
+import { constants } from "../../lib/constants.js";
+import { useEffect } from "preact/hooks";
 
 export function UploaderLogin() {
+  const isUploader = useUploaderAuthProtected();
+
+  useEffect(() => {
+    if (isUploader) {
+      route(constants.ROUTES.UPLOADER.DASHBOARD, true);
+    }
+  }, [isUploader]);
+
+  if (isUploader) {
+    return null;
+  }
+
   return (
     <MainComponentWrapper useFooter={true}>
       <UploaderAuth />

diff --git a/ui/src/components/uploader/PhotoUploader.jsx b/ui/src/components/uploader/PhotoUploader.jsx
@@ -113,10 +113,12 @@ export function PhotoUploader() {
             value={file}
             onInput={async (e) => {
               setFile(e.target?.files[0]);
-              const dateTime = await tryGetDateTimeFromImage(
+              const currentDateTime = await tryGetDateTimeFromImage(
                 e.target?.files[0]
               );
-              setDateTime(dateTime);
+              if (currentDateTime !== null) {
+                setDateTime(currentDateTime);
+              }
             }}
             className="file-input file-input-bordered w-full"
           />