Skip to content

Conversation

@ebinbellini
Copy link

No description provided.

This adds a profile page and surrounding functionality
Look at dev...feature/profile-page
if you want the git history
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR implements a profile/account page with both frontend and backend components, allowing users to view and update their account information including contact details and study program information.

Key changes:

  • Added React dependencies and TypeScript types for a new frontend profile page
  • Created reusable TextInput component with validation and error handling
  • Implemented backend API endpoints for account management and section/program retrieval

Reviewed changes

Copilot reviewed 16 out of 18 changed files in this pull request and generated 19 comments.

Show a summary per file
File Description
package.json Added React 19 and TypeScript type definitions as dependencies
package-lock.json Locked dependency versions for the added packages
frontend/apply/src/app/styles/textinput.css Styling for the TextInput component with states (focused, disabled, error)
frontend/apply/src/app/icons/*.jsx SVG icon components for person, phone, email, section, program, and number fields
frontend/apply/src/app/globals.css Formatting improvements and added disabled button styles
frontend/apply/src/app/components/TextInput.tsx Reusable form input component supporting text, select, and validation
frontend/apply/src/app/account/page.tsx Main account page with form state management, validation, and API integration
frontend/apply/src/app/account/account.module.css Styling for the account page cards and form layout
backend/backend/views.py Added MyAccountAPIView and SectionsAPIView endpoints, modified imports
backend/backend/urls.py Added URL routes for /api/account/ and /api/sections/ endpoints
backend/backend/serializers.py Added serializers for Section, StudyProgram, and updated MemberSerializer
backend/backend/models.py Updated comment to reflect correct field name (study_program)
.gitignore Added node_modules to ignore list
Comments suppressed due to low confidence (2)

backend/backend/serializers.py:3

  • Import of 'ValidationError' is not used.
from rest_framework.serializers import ModelSerializer, ValidationError

backend/backend/serializers.py:4

  • Import of 'Team' is not used.
from .models import Position, Role, Team, Member, Section, StudyProgram, Application, Reference

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ebinbellini ebinbellini linked an issue Dec 22, 2025 that may be closed by this pull request
@ebinbellini
Copy link
Author

account-page-demo.mp4

Things not covered in this PR:

  • Updating information from Unicore
  • Membership status ("You are a member since 2022." is hardcoded)
  • Spelling FUTF correctly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

My profile page

2 participants