From 513f23311243a271a5c35f0ea113c0c5a9289624 Mon Sep 17 00:00:00 2001
From: tiago-bacelar <pereira.migueltiago@gmail.com>
Date: Tue, 6 Feb 2024 01:29:26 +0000
Subject: [PATCH] Fix edit profile

---
 context/Auth/AuthContext.tsx                  |  6 +--
 layout/Attendee/Profile/Profile.tsx           | 26 +++++-----
 layout/SignUp/components/SignUpForm/index.tsx | 52 +++++++++----------
 3 files changed, 42 insertions(+), 42 deletions(-)

diff --git a/context/Auth/AuthContext.tsx b/context/Auth/AuthContext.tsx
index 492e84a2..8fddf54e 100644
--- a/context/Auth/AuthContext.tsx
+++ b/context/Auth/AuthContext.tsx
@@ -192,7 +192,7 @@ export function AuthProvider({ children }) {
       })
       .catch((errors) => {
         if (errors.response?.data?.error) {
-          setErrors(errors.response.data.error);
+          setErrors("Invalid email or password"); //errors.response.data.error
         } else if (errors.response) {
           setErrors("Request denied by the server");
         } else if (errors.request) {
@@ -216,11 +216,11 @@ export function AuthProvider({ children }) {
     router.push("/").finally(() => setUser(null));
   }
 
-  function editUser(nickname) {
+  function editUser(formData: FormData) {
     setLoading(true);
 
     api
-      .editUser(user.id, nickname)
+      .editUser(user.id, formData)
       .then((at) => {
         setUser((oldUser) => ({ ...oldUser, ...at }));
       })
diff --git a/layout/Attendee/Profile/Profile.tsx b/layout/Attendee/Profile/Profile.tsx
index 404030f0..05290179 100644
--- a/layout/Attendee/Profile/Profile.tsx
+++ b/layout/Attendee/Profile/Profile.tsx
@@ -24,12 +24,12 @@ interface Course {
 function Profile() {
   const { user, editUser } = useAuth() as {
     user: IAttendee;
-    editUser: (username: FormData) => void;
+    editUser: (zz: FormData) => void;
   };
   const [avatar, setAvatar] = useState(null);
   const [editing, setEditing] = useState(false);
-  const [username, setUsername] = useState(user.nickname || "");
-  const [course, setCourse] = useState(user.course.toString() || "");
+  const [name, setName] = useState(user.name);
+  const [course, setCourse] = useState(user.course.toString());
 
   const [courses, setCourses] = useState<Course[]>([{ id: "", name: "None" }]);
 
@@ -72,7 +72,7 @@ function Profile() {
   const handleSubmitForm = (e: React.FormEvent) => {
     e.preventDefault();
     const formData = new FormData();
-    formData.append("attendee[nickname]", username);
+    formData.append("attendee[name]", name);
     formData.append("attendee[course_id]", course);
     formData.append("attendee[avatar]", avatar);
 
@@ -151,23 +151,23 @@ function Profile() {
 
             <div className="w-full sm:w-96">
               <Input
-                text="NAME"
-                id="name"
-                name="name"
-                value={user.name || ""}
+                text="NICKNAME"
+                id="nickname"
+                name="nickname"
+                value={user.nickname}
                 bgColor="primary"
                 fgColor="white"
                 enabled={false}
               />
               <Input
-                text="USERNAME"
-                id="username"
-                name="username"
-                value={username}
+                text="NAME"
+                id="name"
+                name="name"
+                value={name}
                 bgColor="primary"
                 fgColor="white"
                 enabled={editing}
-                onChange={(e) => setUsername(e.currentTarget.value)}
+                onChange={(e) => setName(e.currentTarget.value)}
               />
               <Select
                 text="COURSE"
diff --git a/layout/SignUp/components/SignUpForm/index.tsx b/layout/SignUp/components/SignUpForm/index.tsx
index 270d42e7..110095f7 100644
--- a/layout/SignUp/components/SignUpForm/index.tsx
+++ b/layout/SignUp/components/SignUpForm/index.tsx
@@ -38,7 +38,7 @@ export default function SignUpForm({ courses }) {
 
   const validateNickname = (nickname) => {
     return (
-      nickname.length >= 2 &&
+      nickname.length >= 3 &&
       nickname.length <= 15 &&
       nickname.match(/^[\w\d-_]{3,15}$/)
     );
@@ -57,7 +57,7 @@ export default function SignUpForm({ courses }) {
       updateError("You must have a scanned QR code");
     } else if (!validateNickname(nickname)) {
       updateError(
-        "Your nickname must be between 2 and 15 alphanumeric characters and can only contain underscores and dashes"
+        "Your username must be between 3 and 15 alphanumeric characters, underscores and dashes (no spaces allowed)"
       );
     } else if (!validatePassword(password)) {
       updateError("Your password must be at least 8 characters long");
@@ -81,14 +81,6 @@ export default function SignUpForm({ courses }) {
   return (
     <>
       <Form onSubmit={onFinish}>
-        <Input
-          text="NAME"
-          id="name"
-          name="name"
-          fgColor="white"
-          bgColor="primary"
-          onChange={(e) => updateName(e.currentTarget.value)}
-        />
         <Input
           text="EMAIL"
           id="email"
@@ -99,6 +91,22 @@ export default function SignUpForm({ courses }) {
           bgColor="primary"
           onChange={(e) => updateEmail(e.currentTarget.value)}
         />
+        <PasswordInput
+          text="PASSWORD"
+          id="password"
+          name="password"
+          fgColor="white"
+          bgColor="primary"
+          onChange={(e) => updatePassword(e.currentTarget.value)}
+        />
+        <PasswordInput
+          text="CONFIRM PASSWORD"
+          id="confirm"
+          name="confirm"
+          fgColor="white"
+          bgColor="primary"
+          onChange={(e) => updatePasswordConfirmation(e.currentTarget.value)}
+        />
         <Input
           text="NICKNAME"
           id="nickname"
@@ -107,6 +115,14 @@ export default function SignUpForm({ courses }) {
           bgColor="primary"
           onChange={(e) => updateNickname(e.currentTarget.value)}
         />
+        <Input
+          text="NAME"
+          id="name"
+          name="name"
+          fgColor="white"
+          bgColor="primary"
+          onChange={(e) => updateName(e.currentTarget.value)}
+        />
         <Select
           text="COURSE"
           id="course"
@@ -119,22 +135,6 @@ export default function SignUpForm({ courses }) {
           }))}
           onChange={(e) => updateCourse(e.currentTarget.value)}
         />
-        <PasswordInput
-          text="PASSWORD"
-          id="password"
-          name="password"
-          fgColor="white"
-          bgColor="primary"
-          onChange={(e) => updatePassword(e.currentTarget.value)}
-        />
-        <PasswordInput
-          text="CONFIRM PASSWORD"
-          id="confirm"
-          name="confirm"
-          fgColor="white"
-          bgColor="primary"
-          onChange={(e) => updatePasswordConfirmation(e.currentTarget.value)}
-        />
         <Button
           title={scanning ? "STOP SCANNING" : "SCAN QR"}
           className="h-12 w-full border-quinary bg-quinary text-secondary"