Enhance member profiles and related components

- Create separate MemberProfile struct for more user information (#51)
- Update Member struct to reflect changes in the member table (#51)
- Modify db queries to work with the new member_profile table (#51)
- Update GetMember query to join member and member_profile tables (#51)
- Change UpdateMemberByEmail to UpdateMemberProfileByID (#51)
- Adjust logic in EditMemberProfile to use UpdateMemberProfileByID (#51)
- Add new fields to edit profile template (pronouns, bio, etc) (#51)
- Update member.html template to display new profile information (#51)
- Modify createOrReturnID function to add a member_profile entry (#51)
- Remove unused fields from member table (last_chat, last_search)
- Add necessary indexes and foreign key constraints for member_profile
- Update tests to accommodate changes in member profile handling

Change-Id: I2174db3e70a75a69f83221a9391f99a6d456a09b
Signed-off-by: Ian Meyer <[email protected]>

server.go

@@ -443,6 +443,9 @@ func (s *DiscussService) CreateThreadPost(w http.ResponseWriter, r *http.Request
 }
 
 func (s *DiscussService) EditMemberProfile(w http.ResponseWriter, r *http.Request) {
+	s.logger.DebugContext(r.Context(), "entering EditMemberProfile")
+	defer s.logger.DebugContext(r.Context(), "exiting EditMemberProfile")
+
 	if r.Method != http.MethodPost && r.Method != http.MethodGet {
 		s.renderError(w, http.StatusMethodNotAllowed)
 		return
@@ -492,10 +495,13 @@ func (s *DiscussService) EditMemberProfile(w http.ResponseWriter, r *http.Reques
 	// Get the new photo URL from the form
 	newPhotoURL := r.Form.Get("photo_url")
 	newLocation := r.Form.Get("location")
+	newPreferredName := r.Form.Get("preferred_name")
+	newBio := r.Form.Get("bio")
+	newPronouns := r.Form.Get("pronouns")
 
 	// Update the member's profile
-	err = s.queries.UpdateMemberByEmail(r.Context(), UpdateMemberByEmailParams{
-		Email: user.Email,
+	err = s.queries.UpdateMemberProfileByID(r.Context(), UpdateMemberProfileByIDParams{
+		MemberID: user.ID,
 		PhotoUrl: pgtype.Text{
 			String: parseHTMLStrict(newPhotoURL),
 			Valid:  true,
@@ -504,8 +510,21 @@ func (s *DiscussService) EditMemberProfile(w http.ResponseWriter, r *http.Reques
 			String: parseHTMLStrict(newLocation),
 			Valid:  true,
 		},
+		PreferredName: pgtype.Text{
+			String: parseHTMLStrict(newPreferredName),
+			Valid:  true,
+		},
+		Bio: pgtype.Text{
+			String: parseHTMLStrict(newBio),
+			Valid:  true,
+		},
+		Pronouns: pgtype.Text{
+			String: parseHTMLStrict(newPronouns),
+			Valid:  true,
+		},
 	})
 	if err != nil {
+		s.logger.ErrorContext(r.Context(), "UpdateMemberProfileByID", slog.String("error", err.Error()))
 		s.renderError(w, http.StatusInternalServerError)
 		return
 	}

server_test.go

@@ -1409,7 +1409,7 @@ func (m *MockQueries) ListThreads(ctx context.Context, arg ListThreadsParams) ([
 	}, nil
 }
 
-func (m *MockQueries) UpdateMemberByEmail(ctx context.Context, arg UpdateMemberByEmailParams) error {
+func (m *MockQueries) UpdateMemberProfileByID(ctx context.Context, arg UpdateMemberProfileByIDParams) error {
 	// Mock implementation
 	return nil
 }

sqlc/queries.sql

@@ -21,7 +21,25 @@ SELECT id::bigint, is_admin::boolean FROM createOrReturnID($1);
 SELECT id FROM member WHERE email = $1;
 
 -- name: GetMember :one
-SELECT email, location, id, date_joined, photo_url FROM member WHERE id = $1;
+SELECT
+  m.email,
+  mp.location,
+  m.id,
+  mp.bio,
+  mp.timezone,
+  mp.preferred_name,
+  mp.proper_name,
+  mp.pronouns,
+  m.date_joined,
+  mp.photo_url
+FROM
+  member m
+LEFT JOIN
+  member_profile mp
+ON
+  m.id = mp.member_id
+WHERE
+  m.id = $1;
 
 -- name: ListThreads :many
 SELECT
@@ -146,11 +164,15 @@ FROM thread_post tp LEFT JOIN member m
   ON tp.member_id=m.id
 WHERE tp.id=$1 AND m.id=$2;
 
--- name: UpdateMemberByEmail :exec
-UPDATE member SET
+-- name: UpdateMemberProfileByID :exec
+UPDATE member_profile SET
   photo_url = $2,
-  location = $3
-WHERE email = $1;
+  location = $3,
+  bio = $4,
+  timezone = $5,
+  preferred_name = $6,
+  pronouns = $7
+WHERE member_id = $1;
 
 -- name: UpdateThread :exec
 UPDATE thread SET