Skip to content

DBC22-6874: fix Bulletin header unread count pill#1399

Open
ray-oxd wants to merge 1 commit into
mainfrom
bugfix/DBC22-6874
Open

DBC22-6874: fix Bulletin header unread count pill#1399
ray-oxd wants to merge 1 commit into
mainfrom
bugfix/DBC22-6874

Conversation

@ray-oxd

@ray-oxd ray-oxd commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

📝 Submitter

🔗 JIRA Ticket


✅ Quality Assurance & Requirements

  • Requirements Met: I have confirmed that all acceptance criteria from the JIRA ticket are fulfilled.
  • Tested desktop in local or dev envs
  • Tested mobile in local or dev envs
  • Ran unit tests locally
  • SonarCloud: I have verified that the SonarCloud analysis is clean/passing for this branch.

⚙️ Configuration & Environment

  • New Env Variables: No

🧪 How to Test (if required)

  1. Open DriveBC and note the Bulletins nav item (header unread pill may be missing before this fix).
  2. In CMS, create or edit a Bulletin and click Publish with notifications.
  3. Confirm the Bulletins header item shows an update pill with the correct unread count (e.g. 1).
  4. Repeat with an Advisory and confirm Advisories header pill still works.
  5. Optional: if other bulletins have never been published with notifications (last_notified_at null), confirm they do not suppress the count for notified ones.

🔍 Reviewer Checklist

  • Reviewed code for logic and cleanliness
  • Re-tested desktop/mobile in local or dev envs
  • Verified no new console warnings/errors
  • Confirmed that any new env variables are understood/documented

Count unread items per entry instead of returning zero when the first
list item has no last_notified_at.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant