Redesign the Profile Page, Add Public Profiles

backend/api/profile.py

@@ -66,6 +66,9 @@ def update_profile(
             email=profile.email,
             pronouns=profile.pronouns,
             accepted_community_agreement=profile.accepted_community_agreement,
+            bio=profile.bio,
+            linkedin=profile.linkedin,
+            website=profile.website,
         )
         user = user_svc.create(user, user)
     else:
@@ -75,6 +78,9 @@ def update_profile(
         user.pronouns = profile.pronouns
         user.onyen = onyen
         user.accepted_community_agreement = profile.accepted_community_agreement
+        user.bio = profile.bio
+        user.linkedin = profile.linkedin
+        user.website = profile.website
         user = user_svc.update(user, user)
 
     user_details = user_svc.get(user.pid)

backend/api/user.py

@@ -18,3 +18,13 @@ def search(
 ):
     """Search for users based on a query string which matches against name, onyen, and email address."""
     return user_svc.search(subject, q)
+
+
+@api.get("/{onyen}", tags=["Users"])
+def get_by_onyen(
+    onyen: str,
+    subject: User = Depends(registered_user),
+    user_svc: UserService = Depends(),
+):
+    """Search for one user by their onyen"""
+    return user_svc.get_by_onyen(subject, onyen)

backend/entities/event_registration_entity.py

@@ -107,9 +107,14 @@ def to_flat_model(self) -> PublicUser:
         """
         return PublicUser(
             id=self.user_id,
+            onyen=self.user.onyen,
             first_name=self.user.first_name,
             last_name=self.user.last_name,
             pronouns=self.user.pronouns,
             email=self.user.email,
             github_avatar=self.user.github_avatar,
+            github=self.user.github,
+            bio=self.user.bio,
+            linkedin=self.user.linkedin,
+            website=self.user.website,
         )

backend/entities/user_entity.py

@@ -47,6 +47,12 @@ class UserEntity(EntityBase):
     accepted_community_agreement: Mapped[bool] = mapped_column(
         Boolean, nullable=False, default=False
     )
+    # Bio of the user
+    bio: Mapped[str | None] = mapped_column(String(), nullable=True)
+    # LinkedIn URL for the user
+    linkedin: Mapped[str | None] = mapped_column(String(), nullable=True)
+    # Website of the user
+    website: Mapped[str | None] = mapped_column(String(), nullable=True)
 
     # All of the roles for the given user.
     # NOTE: This field establishes a many-to-many relationship between the users and roles table.
@@ -92,6 +98,9 @@ def from_model(cls, model: User) -> Self:
             github_id=model.github_id,
             github_avatar=model.github_avatar,
             accepted_community_agreement=model.accepted_community_agreement,
+            bio=model.bio,
+            linkedin=model.linkedin,
+            website=model.website,
         )
 
     def to_model(self) -> User:
@@ -113,6 +122,9 @@ def to_model(self) -> User:
             github_avatar=self.github_avatar,
             pronouns=self.pronouns,
             accepted_community_agreement=self.accepted_community_agreement,
+            bio=self.bio,
+            linkedin=self.linkedin,
+            website=self.website,
         )
 
     def update(self, model: User) -> None:
@@ -133,13 +145,21 @@ def update(self, model: User) -> None:
         self.github_id = model.github_id or None
         self.github_avatar = model.github_avatar or ""
         self.accepted_community_agreement = model.accepted_community_agreement
+        self.bio = model.bio
+        self.linkedin = model.linkedin
+        self.website = model.website
 
     def to_public_model(self) -> PublicUser:
         return PublicUser(
             id=self.id,
+            onyen=self.onyen,
             first_name=self.first_name,
             last_name=self.last_name,
             pronouns=self.pronouns,
             email=self.email,
             github_avatar=self.github_avatar,
+            github=self.github,
+            bio=self.bio,
+            linkedin=self.linkedin,
+            website=self.website,
         )

backend/migrations/versions/3da4a0776787_migration_for_484_profile_page_m3.py

@@ -0,0 +1,33 @@
+"""Migration for 484-profile-page-m3
+
+Revision ID: 3da4a0776787
+Revises: 909fe23fc93f
+Create Date: 2024-07-23 18:32:25.724472
+"""
+
+from alembic import op
+import sqlalchemy as sa
+
+
+revision = "3da4a0776787"
+down_revision = "909fe23fc93f"
+branch_labels = None
+depends_on = None
+
+
+def upgrade() -> None:
+    op.add_column(
+        "user", sa.Column("bio", sa.String(), nullable=False, server_default="")
+    )
+    op.add_column(
+        "user", sa.Column("linkedin", sa.String(), nullable=False, server_default="")
+    )
+    op.add_column(
+        "user", sa.Column("website", sa.String(), nullable=False, server_default="")
+    )
+
+
+def downgrade() -> None:
+    op.drop_column("user", "website")
+    op.drop_column("user", "linkedin")
+    op.drop_column("user", "bio")

backend/models/public_user.py

@@ -17,8 +17,13 @@ class PublicUser(BaseModel):
     """
 
     id: int | None
+    onyen: str
     first_name: str
     last_name: str
     pronouns: str
     email: str
     github_avatar: str | None = None
+    github: str | None = None
+    bio: str | None = None
+    linkedin: str | None = None
+    website: str | None = None

backend/models/user.py

@@ -36,6 +36,9 @@ class User(UserIdentity, BaseModel):
     github_id: int | None = None
     github_avatar: str | None = None
     accepted_community_agreement: bool = False
+    bio: str | None = None
+    linkedin: str | None = None
+    website: str | None = None
 
 
 class NewUser(User, BaseModel):
@@ -63,3 +66,6 @@ class ProfileForm(BaseModel):
     pronouns: str
     email: str
     accepted_community_agreement: bool = False
+    bio: str | None = None
+    linkedin: str | None = None
+    website: str | None = None

backend/services/user.py

@@ -7,7 +7,7 @@
 from sqlalchemy import select, or_, func, cast, String
 from sqlalchemy.orm import Session
 from ..database import db_session
-from ..models import User, UserDetails, Paginated, PaginationParams
+from ..models import User, UserDetails, Paginated, PaginationParams, PublicUser
 from ..entities import UserEntity
 from .exceptions import ResourceNotFoundException
 from .permission import PermissionService
@@ -68,6 +68,25 @@ def get_by_id(self, id: int) -> User:
 
         return user_entity.to_model()
 
+    def get_by_onyen(self, subject: User, onyen: str) -> PublicUser:
+        """Get a User by their onyen.
+
+        Args:
+            onyen: The onyen of the user.
+
+        Returns:
+            PublicUser
+
+        Raises:
+            ResourceNotFoundException if the User ID is not found
+        """
+        user_query = select(UserEntity).where(UserEntity.onyen == onyen)
+        user_entity = self._session.scalars(user_query).one_or_none()
+        if user_entity is None:
+            raise ResourceNotFoundException(f"User with {id} not found")
+
+        return user_entity.to_public_model()
+
     def search(self, _subject: User, query: str) -> list[User]:
         """Search for users by their name, onyen, email.
 
@@ -84,7 +103,7 @@ def search(self, _subject: User, query: str) -> list[User]:
             UserEntity.last_name.ilike(f"%{query}%"),
             UserEntity.onyen.ilike(f"%{query}%"),
             UserEntity.email.ilike(f"%{query}%"),
-            cast(UserEntity.pid, String).ilike(f"%{query}%")
+            cast(UserEntity.pid, String).ilike(f"%{query}%"),
         )
         statement = statement.where(criteria).limit(10)
         entities = self._session.execute(statement).scalars()

backend/test/services/event/event_test_data.py

@@ -67,10 +67,13 @@
     organizers=[
         PublicUser(
             id=root.id,
+            onyen=root.onyen,
             first_name=root.first_name,
             last_name=root.last_name,
             pronouns=root.pronouns,
             email=root.email,
+            linkedin=root.linkedin,
+            website=root.website,
         )
     ],
 )
@@ -98,10 +101,13 @@
     organizers=[
         PublicUser(
             id=user.id,
+            onyen=user.onyen,
             first_name=user.first_name,
             last_name=user.last_name,
             pronouns=user.pronouns,
             email=user.email,
+            linkedin=user.linkedin,
+            website=user.website,
         ),
     ],
 )
@@ -118,17 +124,23 @@
     organizers=[
         PublicUser(
             id=user.id,
+            onyen=user.onyen,
             first_name=user.first_name,
             last_name=user.last_name,
             pronouns=user.pronouns,
             email=user.email,
+            linkedin=user.linkedin,
+            website=user.website,
         ),
         PublicUser(
             id=ambassador.id,
+            onyen=ambassador.onyen,
             first_name=ambassador.first_name,
             last_name=ambassador.last_name,
             pronouns=ambassador.pronouns,
             email=ambassador.email,
+            linkedin=ambassador.linkedin,
+            website=ambassador.website,
         ),
     ],
 )
@@ -156,24 +168,33 @@
     organizers=[
         PublicUser(
             id=user.id,
+            onyen=user.onyen,
             first_name=user.first_name,
             last_name=user.last_name,
             pronouns=user.pronouns,
             email=user.email,
+            linkedin=user.linkedin,
+            website=user.website,
         ),
         PublicUser(
             id=ambassador.id,
+            onyen=user.onyen,
             first_name=ambassador.first_name,
             last_name=ambassador.last_name,
             pronouns=ambassador.pronouns,
             email=ambassador.email,
+            linkedin=ambassador.linkedin,
+            website=ambassador.website,
         ),
         PublicUser(
             id=root.id,
+            onyen=user.onyen,
             first_name=root.first_name,
             last_name=root.last_name,
             pronouns=root.pronouns,
             email=root.email,
+            linkedin=root.linkedin,
+            website=root.website,
         ),
     ],
 )
@@ -190,10 +211,13 @@
     organizers=[
         PublicUser(
             id=user.id,
+            onyen=user.onyen,
             first_name=user.first_name,
             last_name=user.last_name,
             pronouns=user.pronouns,
             email=user.email,
+            linkedin=user.linkedin,
+            website=user.website,
         ),
     ],
 )

frontend/src/app/academics/academics-admin/section/section-editor/section-editor.component.ts

@@ -203,11 +203,16 @@ export class SectionEditorComponent {
         .map((staff) => {
           return {
             id: staff.user_id!,
+            onyen: '',
             first_name: staff.first_name,
             last_name: staff.last_name,
             pronouns: '',
             email: '',
-            github_avatar: ''
+            github_avatar: '',
+            github: '',
+            bio: '',
+            linkedin: '',
+            website: ''
           };
         }) ?? [];
   }

frontend/src/app/event/widgets/event-card/event-card.widget.html

@@ -31,23 +31,23 @@
   <mat-card-content>
     <mat-chip-set>
       <!-- Organization chip -->
-      <mat-chip
+      <mat-chip-row
         id="organization-chip"
         [routerLink]="'/organizations/' + event.organization_slug">
         <img
           class="profile-icon"
           matChipAvatar
           [src]="event.organization_icon" />
         {{ event.organization_name }}
-      </mat-chip>
+      </mat-chip-row>
 
       @if (event.organizers.length > 0) {
       <mat-divider class="vertical-divider" [vertical]="true" />
       }
 
       <!-- Organizer Chips -->
       @for (organizer of event.organizers; track organizer.id) {
-      <mat-chip>
+      <mat-chip-row [routerLink]="'/profile/' + organizer.onyen">
         @if (organizer.github_avatar && organizer.github_avatar !== '') {
         <img
           class="profile-icon"
@@ -56,7 +56,7 @@
         }
 
         {{ organizer.first_name }} {{ organizer.last_name }}
-      </mat-chip>
+      </mat-chip-row>
       }
     </mat-chip-set>
     <p class="description">