fix: save bio in profile object instead of in basic_info trait

Author: rakibansary

src/routes/BuildMyProfile/index.jsx

@@ -46,6 +46,7 @@ import { getTraits, scrollToTop, isProfileFormDataEmpty } from "utils/";
 import moment from "moment";
 import _ from "lodash";
 import { createTraits, updateTraits } from "services/traits";
+import { updateMemberData } from "services/memberData";
 
 const formatDate = (date) => {
   let ret = new Date(
@@ -59,12 +60,13 @@ const BuildMyProfile = () => {
   const authUser = useSelector((state) => state.authUser);
   const [isLoading, setIsLoading] = useState(false);
   const [myProfileData, setMyProfileData] = useState({});
+  const [bio, setBio] = useState("");
+
   // form states
   const [formData, setFormData] = useState({
     title: "",
-    bio: "",
   });
-  const { title, bio } = formData;
+  const { title } = formData;
   // get an empty job item
   const emptyJob = () => ({
     company: "",
@@ -190,6 +192,7 @@ const BuildMyProfile = () => {
     getAuthUserProfile()
       .then((result) => {
         setMyProfileData(result);
+        setBio(result.description);
       })
       .catch((e) => {
         // eslint-disable-next-line no-console
@@ -218,11 +221,10 @@ const BuildMyProfile = () => {
         // fill title and bio to state
         if (basicInfoValue) {
           // Using shortBio as title has to do with v3 using this mapping
-          const { description, shortBio: title } = basicInfoValue;
+          const { shortBio: title } = basicInfoValue;
           setFormData((formData) => ({
             ...formData,
             title: title || "",
-            bio: description || "",
           }));
         }
         if (workExpValue) {
@@ -273,16 +275,15 @@ const BuildMyProfile = () => {
     scrollToTop();
   }, []);
 
-  // save title / bio
-  const getTitleAndBioData = (basicInfo) => {
-    const { title, bio } = formData;
+  // save title
+  const getTitle = (basicInfo) => {
+    const { title } = formData;
     // mapped data
     let data = {
       shortBio: title,
-      description: bio,
     };
     // check if basic info already exists. if so, update(put data). otherwise, post data.
-    if (basicInfo == null && isProfileFormDataEmpty("bio", data)) {
+    if (basicInfo == null && isProfileFormDataEmpty("title", data)) {
       return {
         action: "create",
         data: {
@@ -293,10 +294,8 @@ const BuildMyProfile = () => {
           },
         },
       };
-      // return addMyTitleAndBio(authUser.handle, data);
     } else {
       if (isProfileFormDataEmpty("bio", data)) {
-        // return updateMyTitleAndBio(authUser.handle, basicInfo, data);
         return {
           action: "update",
           data: {
@@ -451,8 +450,6 @@ const BuildMyProfile = () => {
     );
   };
 
-  // reach router, navigate programmatically
-  const navigate = useNavigate();
   // on submit, save form and then navigate
   const handleSubmit = async (e) => {
     if (!canSubmit()) {
@@ -481,7 +478,7 @@ const BuildMyProfile = () => {
     const functions = [
       {
         val: basicInfoTraits,
-        func: getTitleAndBioData,
+        func: getTitle,
       },
       {
         val: workExperience,
@@ -533,6 +530,10 @@ const BuildMyProfile = () => {
       }
     }
 
+    if (myProfileData.description != bio && bio != null && bio.trim().length) {
+      await updateMemberData(authUser.handle, { description: bio });
+    }
+
     setIsLoading(false);
     window.location.href = config.TOPCODER_COMMUNITY_WEBSITE_URL + "/home";
   };
@@ -598,9 +599,7 @@ const BuildMyProfile = () => {
                   placeholder={"Enter your bio"}
                   value={bio}
                   name="bio"
-                  onChange={(e) =>
-                    handleInputChange(e.target.name, e.target.value)
-                  }
+                  onChange={(e) => setBio(e.target.value)}
                 />
               </FormField>
             </div>

src/utils/index.js

@@ -23,11 +23,9 @@ export function scrollToTop() {
  */
 export function isProfileFormDataEmpty(type, data) {
   switch (type) {
-    case "bio":
-      let { shortBio, description } = data;
-      return (
-        (shortBio && shortBio.length) || (description && description.length)
-      );
+    case "title":
+      let { shortBio } = data;
+      return shortBio && shortBio.length;
     case "work":
       let { cityTown, company, position, timePeriodFrom, timePeriodTo } = data;
       let response =