fix: remove edit profile button

bxiang commented

2026-10-03 20:43:23 +00:00

Member

Description

Removes the Edit Profile button from the Profile page because it no longer provides any functionality. I removed the button from ProfileHeader, along with the unused onEdit prop, edit handler, and related ref. I also added a test to verify that the Edit Profile button is no longer rendered.

(#30)

Closes # 30

Type of change

Type: Bug fix / Test coverage

Testing

I added/updated tests.
I tested my changes locally.
- I tested my changes with a laptop-size screen.
- I tested my changes with a phone-size screen.
- I tested my changes with a tablet-size screen.
I tested on the kennel preview deployment for this pull request.

Screenshots/Recordings

Checklist

My code follows the project's existing style.
I have reviewed my changes.
Only the necessary files and lines have been changed.
This PR is ready for review.

## Description Removes the Edit Profile button from the Profile page because it no longer provides any functionality. I removed the button from `ProfileHeader`, along with the unused `onEdit` prop, edit handler, and related ref. I also added a test to verify that the Edit Profile button is no longer rendered. ## Related Issue (https://git.cmu.dev/ScottyLabs/study/issues/30) Closes # 30 ## Type of change **Type:** Bug fix / Test coverage ## Testing - [x] I added/updated tests. - [x] I tested my changes locally. - [x] I tested my changes with a laptop-size screen. - [x] I tested my changes with a phone-size screen. - [x] I tested my changes with a tablet-size screen. - [x] I tested on the kennel preview deployment for this pull request. ## Screenshots/Recordings  ## Checklist - [x] My code follows the project's existing style. - [x] I have reviewed my changes. - [x] Only the necessary files and lines have been changed. - [x] This PR is ready for review.

phone.png

173 KiB

#30.png

371 KiB

bxiang added 2 commits

2026-10-03 20:43:23 +00:00

fix: remove edit profile button 419255b7d0

test: verify edit profile button is removed

Deploy to Railway Dev / Deploy to Dev (push) Successful in 2m8s

Details

kennel/build skipped: no services or static sites defined

Details

Test / Unit and integration tests (push) Successful in 2m5s

Details

Test / Unit and integration tests (pull_request) Successful in 2m0s

Details

7e775f0ddf

bxiang requested reviews from annadavi, mishag, cseluzhy

2026-10-03 20:43:50 +00:00

bxiang changed title from ~~issue-30-remove-edit-profile-button~~ to fix: remove edit profile button

2026-10-03 21:03:41 +00:00

bxiang changed title from ~~fix: remove edit profile button~~ to fix: remove edit profile button #30

2026-10-03 21:04:04 +00:00

bxiang changed title from ~~fix: remove edit profile button #30~~ to fix: remove edit profile button

2026-10-03 21:04:09 +00:00

annadavi commented

2026-10-04 02:57:24 +00:00

Member

For future reference, you only need to put the issue number once, next to the existing "Closes #", but don't put a space between the "#" and the number. Also, make sure you have actually completed each check before marking the box (you couldn't have tested with kennel yet, so that can be left unchecked for now).

annadavi approved these changes

2026-10-04 02:58:31 +00:00

annadavi left a comment

Be sure to update your branch with merge or rebase from the base branch (dropdown below on the right) before creating the merge commit into ui_change!

sudipi scheduled this pull request to auto merge when all checks succeed

2026-10-24 20:51:17 +00:00

sudipi scheduled this pull request to auto merge when all checks succeed

2026-10-24 20:54:16 +00:00

bxiang canceled auto merging this pull request when all checks succeed

2026-10-24 21:13:26 +00:00

annadavi added 1 commit

2026-10-26 02:44:19 +00:00

fix: merge ui_change into remove-edit-profile-button

nixbot/nix-build 0 attributes built

Details

kennel/build build succeeded