feat: add 'Show read notifications' setting#2488

Merged

afonsojramos merged 20 commits intomainfrom

review-open-issues

Jan 13, 2026

Commits on Jan 12, 2026

feat: add 'Show read notifications' setting (#708)
afonsojramos
committed
fix: hide mark as read button on read notifications
afonsojramos
committed
fix: fetch read notifications from API when setting enabled
afonsojramos
committed
test: update Settings snapshot for showReadNotifications default
afonsojramos
committed
refactor: address PR feedback for showReadNotifications
afonsojramos
committed
refactor: move mark as done button to far right
afonsojramos
committed
refactor: address PR review feedback from setchy
afonsojramos
committed
style: fix formatting in NotificationSettings test
afonsojramos
committed
refactor: reorder hover group buttons to keep chevron as right-most item

setchy
authored and
afonsojramos
committed
refactor: handle mark as done limitation when fetching read notifications
afonsojramos
committed
fix: provide complete AppContextState defaults in test utils
afonsojramos
committed
fix: correct mock variable references in tests after rebase
afonsojramos
committed
test: update Settings snapshot for tooltip testid change
afonsojramos
committed
test: update login route snapshots for primer ID changes
afonsojramos
committed
test: add coverage for fetchReadNotifications fallback behavior
afonsojramos
committed
fix: keep notifications in list when fetchReadNotifications is enabled
afonsojramos
committed
test: achieve 100% coverage for useNotifications hook
afonsojramos
committed
fix: stabilize snapshot tests by normalizing React auto-generated IDs
afonsojramos
committed
ci: trigger workflow
afonsojramos
committed
test: update snapshots after rebase on main
afonsojramos
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: add 'Show read notifications' setting#2488

feat: add 'Show read notifications' setting#2488
afonsojramos merged 20 commits intomainfrom
review-open-issues

Commits on Jan 12, 2026

feat: add 'Show read notifications' setting (#708)

fix: hide mark as read button on read notifications

fix: fetch read notifications from API when setting enabled

test: update Settings snapshot for showReadNotifications default

refactor: address PR feedback for showReadNotifications

refactor: move mark as done button to far right

refactor: address PR review feedback from setchy

style: fix formatting in NotificationSettings test

refactor: reorder hover group buttons to keep chevron as right-most item

refactor: handle mark as done limitation when fetching read notifications

fix: provide complete AppContextState defaults in test utils

fix: correct mock variable references in tests after rebase

test: update Settings snapshot for tooltip testid change

test: update login route snapshots for primer ID changes

test: add coverage for fetchReadNotifications fallback behavior

fix: keep notifications in list when fetchReadNotifications is enabled

test: achieve 100% coverage for useNotifications hook

fix: stabilize snapshot tests by normalizing React auto-generated IDs

ci: trigger workflow

test: update snapshots after rebase on main