ocervera_ui_change #46

Merged

annadavi merged 7 commits from ocervera_ui_change into ui_change 2026-10-26 02:22:28 +00:00

Conversation 7 Commits 7 Files changed 3 +20 -18

ocervera commented 2026-09-26 22:02:24 +00:00

Member

Copy link

Description

added day of the week besides date on the cards

Related Issue

Closes #22

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.
• [ X ] 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.

## Description added day of the week besides date on the cards ## Related Issue Closes #22 ## 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. - [ X ] 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.

Screenshot 2026-10-03 162918.png

97 KiB

ocervera added 2 commits 2026-09-26 22:02:24 +00:00

Show the weekday alongside group dates. ... 821d7522f7

Co-authored-by: Cursor <cursoragent@cursor.com>

commit f4c0e55de0

ocervera requested reviews from annadavi, mishag, cseluzhy 2026-09-26 22:03:02 +00:00

annadavi force-pushed ocervera_ui_change from f4c0e55de0 to 1564dd4c87 2026-09-28 03:03:54 +00:00 Compare

annadavi commented 2026-09-28 03:04:16 +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>

annadavi commented 2026-10-04 02:32:37 +00:00

Member

Copy link

For future reference, the "- [ ] " does checkboxes in markdown, and putting an X inside without spaces will show a checked checkbox ("- [X] "). The spaces you left in yours are why it doesn't show an actual checkbox.

For future reference, the "- [ ] <item>" does checkboxes in markdown, and putting an X inside without spaces will show a checked checkbox ("- [X] <item>"). The spaces you left in yours are why it doesn't show an actual checkbox.

annadavi approved these changes 2026-10-04 02:32:46 +00:00

Dismissed

annadavi commented 2026-10-04 02:33:55 +00:00

Member

Copy link

Be sure to update the branch with rebase or merge using the dropdown below before creating the merge commit into ui_change.

Be sure to update the branch with rebase or merge using the dropdown below before creating the merge commit into ui_change.

ocervera added 4 commits 2026-10-24 17:54:58 +00:00

Show the weekday alongside group dates. ... 821d7522f7

Co-authored-by: Cursor <cursoragent@cursor.com>

commit f4c0e55de0

Show a single line when nobody is blocked. ... fa5e11f3bf

Co-authored-by: Cursor <cursoragent@cursor.com>

Merge remote ocervera_ui_change before pushing the empty-state fix. 147ae3d391

annadavi added 1 commit 2026-10-26 02:15:08 +00:00

Merge branch 'ui_change' into ocervera_ui_change d825e09429

scottylabs-bot commented 2026-10-26 02:16:32 +00:00

Owner

Copy link

Kennel Deployments

Torn down at 2026-10-26 02:22:28 UTC.

<!-- kennel-deployment --> ### Kennel Deployments Torn down at 2026-10-26 02:22:28 UTC.

annadavi approved these changes 2026-10-26 02:22:09 +00:00

annadavi commented 2026-10-26 02:22:24 +00:00

Member

Copy link

In the future, make separate branches and PRs for each issue. I'm merging this now just so that we can finalize the new UI.

In the future, make separate branches and PRs for each issue. I'm merging this now just so that we can finalize the new UI.

annadavi merged commit d825e09429 into ui_change 2026-10-26 02:22:28 +00:00

annadavi deleted branch ocervera_ui_change 2026-10-26 02:22:28 +00:00

Sign in to join this conversation.

Reviewers

No reviewers

cseluzhy

mishag

annadavi

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

3 participants

Notifications

Subscribe

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!46

No description provided.