-
Notifications
You must be signed in to change notification settings - Fork 0
Profile page frontend and backend #69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
This adds a profile page and surrounding functionality Look at dev...feature/profile-page if you want the git history
There was a problem hiding this 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.
account-page-demo.mp4Things not covered in this PR:
|
No description provided.