Skip to content

#348 Add edit user form to ARO#667

Merged
cryolins merged 6 commits intomainfrom
peilinh/aro-edit-user-form
Feb 7, 2026
Merged

#348 Add edit user form to ARO#667
cryolins merged 6 commits intomainfrom
peilinh/aro-edit-user-form

Conversation

@cryolins
Copy link
Contributor

@cryolins cryolins commented Feb 5, 2026

Add edit user form to ARO

Closes #348.
Added a form at /profile/settings where the ARO user can edit their information. There is a profile picture upload button and a form that the user can edit (disabled until the edit button is clicked to prevent accidental edits).

New Changes

  • profile-form.tsx and profile-form.css files added under ARO components/profile
  • ARO's App.tsx modified to have the new page in the routes

Testing

Explain tests that you ran to verify code functionality.

  • I have unit-tested this PR. Otherwise, explain why it cannot be unit-tested.
  • I have tested this PR on a board if the code will run on a board (Only required for firmware developers).
  • I have tested this PR by running the ARO website (Only required if the code will impact the ARO website).
  • I have tested this PR by running the MCC website (Only required if the code will impact the MCC website).
  • I have included screenshots of the tests performed below.
image

Outstanding Changes

This form should be hooked up to user functionality when that gets implemented later on. The necessity of profile pictures can also be decided :)

@cryolins cryolins changed the title Add edit user form to ARO #348 Add edit user form to ARO Feb 5, 2026
@cryolins cryolins mentioned this pull request Feb 5, 2026
5 tasks
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
camspec
🥇
29
▀▀▀▀
3d 16m
155
▀▀▀▀▀
proprogrammer504
🥈
17
▀▀
17h 31m
44
Adityya-K
🥉
15
▀▀
5d 1h 12m
▀▀
50
▀▀
Syzygicality
11
18h 48m
37
kepler452b123
2
3d 5h 7m
1
joannalauu
2
4d 2h 46m
10
c4bae
1
13d 1h 35m
▀▀▀▀
2
panthpatel2016
1
4h 30m
6

⚡️ Pull request stats

Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

Looks great for the non-functional side, just a few changes would be nice. Also make sure your files end with a newline

@cryolins
Copy link
Contributor Author

cryolins commented Feb 7, 2026

new screenshot
image

camspec
camspec previously approved these changes Feb 7, 2026
Copy link
Member

@camspec camspec left a comment

Choose a reason for hiding this comment

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

lgtm!

@cryolins cryolins merged commit f7d9bb9 into main Feb 7, 2026
49 of 51 checks passed
@cryolins cryolins deleted the peilinh/aro-edit-user-form branch February 7, 2026 21:51
@github-actions github-actions bot added this to the 26W milestone Feb 7, 2026
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.

Add form to allow an ARO user to update some information

2 participants