[Feature:Notifications] Use Course Title on Home Page#12288
Merged
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #12288 +/- ##
============================================
- Coverage 21.69% 21.69% -0.01%
- Complexity 9617 9618 +1
============================================
Files 268 268
Lines 36115 36120 +5
Branches 478 478
============================================
+ Hits 7836 7837 +1
- Misses 27805 27809 +4
Partials 474 474
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Member
williamjallen
left a comment
There was a problem hiding this comment.
This seems like something you could add Cypress tests for easily. Adding a test would significantly improve our confidence that this front-page feature will continue to work in the future.
Fixed issues with merge conflict and used a ternary expression to reduce code duplication
williamjallen
approved these changes
Jan 20, 2026
Contributor
Author
|
@williamjallen Thank you for the approval. I agree with your Cypress suggestion and will add it to a comprehensive PR that addresses #11908. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why is this Change Important & Necessary?
Currently, the notifications on the home page use the course code (example: CSCI1200), instead of the course title (example: Data Structures). Courses are more identifiable by their titles, rather than their codes.
What is the New Behavior?
Course titles are now displayed by default, and course codes are only displayed if the title has not been set.
In this image, notifications use a course's title, and code, based on whether it is defined or not.
What steps should a reviewer take to reproduce or test the bug or new feature?
Verify that clicking on course title/code is still clickable and takes you to the course's notification page. Set course titles (via course settings) and ensure they update on the home page. Verify that the course notifications page is unchanged.
Automated Testing & Documentation
N / A
Other information
This is not a breaking change. I plan to create a followup to this PR with course color-coded notifications.