ScottyLabs/study
Watch
16
0
Fork
You've already forked study
 1
Code Issues 34 Pull requests 3 Projects 1 Releases Packages Wiki Activity Actions

Prioritize account details on profile page #38

Closed
kenneth4 wants to merge 2 commits from feature/28-account-details-priority into ui_change
pull from: feature/28-account-details-priority
merge into: ScottyLabs:ui_change
Conversation 3 Commits 2 Files changed 3 +111 -73
kenneth4 commented 2026-09-26 21:08:11 +00:00
Member
Copy link

Description

Place the account details above course numbers and blocked users.

Closes #28

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Test coverage
  • Documentation

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 dev app (dev.cmustudy.com)

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 Place the account details above course numbers and blocked users. Closes #28 ## Type of change - [ ] Bug fix - [x] New feature - [ ] Refactor / code improvement - [ ] Test coverage - [ ] Documentation ## 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. - [ ] I tested my changes with a tablet-size screen. - [ ] I tested on the dev app (dev.cmustudy.com) ## 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.
Prioritize account details on profile page
Some checks failed
kennel/build skipped: no services or static sites defined
Details
Deploy to Railway Dev / Deploy to Dev (push) Has been cancelled
Details
Test / Unit and integration tests (push) Successful in 1m45s
Details
Test / Unit and integration tests (pull_request) Successful in 1m44s
Details
5c62051fd5
Unit Tests
Some checks failed
kennel/build skipped: no services or static sites defined
Details
Deploy to Railway Dev / Deploy to Dev (push) Has been cancelled
Details
Test / Unit and integration tests (push) Has been cancelled
Details
Test / Unit and integration tests (pull_request) Has been cancelled
Details
1d8df4dbf0
annadavi force-pushed feature/28-account-details-priority from 1d8df4dbf0
Some checks failed
kennel/build skipped: no services or static sites defined
Details
Deploy to Railway Dev / Deploy to Dev (push) Has been cancelled
Details
Test / Unit and integration tests (push) Has been cancelled
Details
Test / Unit and integration tests (pull_request) Has been cancelled
Details
to 5752f6db7e
All checks were successful
kennel/build skipped: no services or static sites defined
Details
Test / Unit tests (push) Successful in 1m32s
Details
Test / Integration tests (push) Successful in 1m37s
Details
Test / Deploy to Production (push) Has been skipped
Details
Test / Deploy to Dev (push) Successful in 2m17s
Details
2026-09-27 15:15:15 +00:00 Compare
annadavi force-pushed feature/28-account-details-priority from 5752f6db7e
All checks were successful
kennel/build skipped: no services or static sites defined
Details
Test / Unit tests (push) Successful in 1m32s
Details
Test / Integration tests (push) Successful in 1m37s
Details
Test / Deploy to Production (push) Has been skipped
Details
Test / Deploy to Dev (push) Successful in 2m17s
Details
to 8910d09b96
All checks were successful
kennel/build skipped: no services or static sites defined
Details
Test / Unit tests (push) Successful in 1m24s
Details
Test / Integration tests (push) Successful in 1m33s
Details
Test / Deploy to Production (push) Has been skipped
Details
Test / Deploy to Dev (push) Successful in 1m34s
Details
2026-09-28 03:01:46 +00:00 Compare
annadavi commented 2026-09-28 03:02:23 +00:00
Member
Copy link

I just added a PR template to the repo. In the future, it will automatically populate new PRs. Since this PR was made before the template existed, please just edit the description, copy the following template into it, and fill in the template:

## Description



## Related Issue


Closes #

## Type of change

- [ ] Bug fix
- [ ] New feature
- [ ] Refactor / code improvement
- [ ] Test coverage
- [ ] Documentation

## 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 dev app (dev.cmustudy.com)

## 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.
I just added a PR template to the repo. In the future, it will automatically populate new PRs. Since this PR was made before the template existed, please just edit the description, copy the following template into it, and fill in the template: <pre> ## Description <!-- Please provide a description of the issue, the changes you made to fix it, and why they work. It is expected that you understand why your changes work and if you do not understand why at least say as much so a maintainer knows how much to value the PR. No AI slop. --> ## Related Issue <!-- Link the issue this PR addresses, if applicable. --> Closes # ## Type of change - [ ] Bug fix - [ ] New feature - [ ] Refactor / code improvement - [ ] Test coverage - [ ] Documentation ## 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 dev app (dev.cmustudy.com) ## Screenshots/Recordings <!-- If this includes a UI change, attach screenshots on both laptop and mobile. --> ## 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. </pre>
kenneth4 commented 2026-10-03 20:15:24 +00:00
Author
Member
Copy link

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Test coverage
  • Documentation

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 dev app (dev.cmustudy.com)

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.
## Type of change - [ ] Bug fix - [x] New feature - [ ] Refactor / code improvement - [ ] Test coverage - [ ] Documentation ## 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. - [ ] I tested on the dev app (dev.cmustudy.com) ## 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.
annadavi commented 2026-10-03 22:05:39 +00:00
Member
Copy link

Can you explain the reason for all the deletions? And in general what exactly you changed and why?

Can you explain the reason for all the deletions? And in general what exactly you changed and why?
annadavi closed this pull request 2026-10-25 22:52:17 +00:00
Some checks are pending
kennel/build skipped: no services or static sites defined
Required
Details
Test / Unit tests (push) Successful in 1m24s
Details
Test / Integration tests (push) Successful in 1m33s
Details
Test / Deploy to Production (push) Has been skipped
Details
Test / Deploy to Dev (push) Successful in 1m34s
Details
Test / Unit tests (pull_request)
Required
Test / Integration tests (pull_request)
Required
kennel/deploy
Required

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
annadavi
mishag
cseluzhy
Labels
Clear labels   
first-task
  
needs/docs
Must have documentation written

  
needs/research
Discuss the ideal implementation with the tech lead

  
needs/upstream
Needs a contribution to an upstream project

  
type/bug
Something not working as intended

  
type/external
Lives outside of code

  
type/feature
New functionality

  
type/refactor
Improving existing code

  
urgency
high
 This is currently causing issues

  
urgency
immediate
 This is making a project or feature unusable

  
urgency
low
 This can happen whenever

  
urgency
medium
 This is expected to cause issues soon

No labels
first-task
needs/docs
needs/research
needs/upstream
type/bug
type/external
type/feature
type/refactor
urgency
high
urgency
immediate
urgency
low
urgency
medium
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
ScottyLabs/study!38
No description provided.