Skip to content

DBC22-6585: enhancement advisory/bulletin publish options for new updates#1371

Open
bcgov-brwang wants to merge 1 commit into
mainfrom
feature/DBC22-6585
Open

DBC22-6585: enhancement advisory/bulletin publish options for new updates#1371
bcgov-brwang wants to merge 1 commit into
mainfrom
feature/DBC22-6585

Conversation

@bcgov-brwang

@bcgov-brwang bcgov-brwang commented Jun 9, 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: Does this PR require new environment variables? (Yes/No)

    If yes, please list them here and ensure they are added to secret manager, the .env.example. and the Vault by an STA.

🧪 How to Test (if required)

Advisory/Bulletin Update Notification Verification

Scenario 1: Highlighting and Numbered Pill Behavior

  • Navigate to dev.drivebc.ca.
  • Ensure all advisories and bulletins have been read and that no numbered pill is displayed on the Advisories or Bulletins header.
  • Open the CMS admin page:
    https://dev.drivebc.ca/drivebc-cms/cms/advisory/, or
    https://dev.drivebc.ca/drivebc-cms/cms/bulletin/
  • Update an advisory or bulletin and select Publish with notifications.
  • Verify that a numbered pill appears on the corresponding Advisories or Bulletins header, indicating that new content is available.
  • Click the Advisories or Bulletins header.
  • Verify that the updated advisory or bulletin is highlighted in the list.
  • Navigate to another page within the application.
  • Verify that:
    The numbered pill disappears after navigating away from the list page.
    The highlighted state is cleared from the updated advisory or bulletin.
    The last Updated time updated.

Scenario 2: Publish Minor Update

  • Update an advisory or bulletin in the CMS admin console.
  • Select Publish minor update.
  • Verify that:
    No numbered pill appears on the Advisories or Bulletins header.
    No advisory or bulletin row is highlighted in the frontend.
    The last Updated time is not updated.

Scenario 3: Advisory/Bulletin Detail Page Updates

  • Navigate to an advisory or bulletin detail page in the frontend.
  • In the CMS admin console, update the same advisory or bulletin.
  • Select Publish with notifications.
  • Verify that:
    The detail page reflects the updated content.
    A new overlay notification appears, informing the user that the advisory or bulletin has been updated.

Scenario 4: Minor Update on Detail Page

  • Navigate to an advisory or bulletin detail page.
  • Update the same advisory or bulletin in the CMS admin console.
  • Select Publish minor update.
  • Verify that:
    The detail page reflects the updated content.
    No overlay notification is displayed.

Scenario 5: Advisory Email Notifications

  • Ensure a registered user has Advisory Notifications enabled.
  • Update an advisory in the CMS admin console.
  • Select Publish with notifications.
  • Verify that the registered user receives an advisory email notification.
    Repeat the update using Publish minor update.
    Verify that no advisory email notification is sent.
    Verify that no email notification is sent if select Publish minor updates.

🔍 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

Comment thread src/frontend/src/Components/advisories/AdvisoriesList.js Fixed
Comment thread src/frontend/src/Components/advisories/AdvisoriesList.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/Components/advisories/AdvisoriesList.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/backend/apps/cms/views.py Fixed
Comment thread src/backend/apps/cms/views.py Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/BulletinsListPage.js Fixed
Comment thread src/frontend/src/pages/BulletinsListPage.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/pages/BulletinsListPage.js Fixed
@bcgov-brwang bcgov-brwang changed the title DBC22-6585: highlighted advisory for new updates DBC22-6585: enhancement advisory/bulletin publish options for new updates Jun 15, 2026
Comment thread src/frontend/src/pages/BulletinsListPage.js Fixed
@bcgov-brwang bcgov-brwang requested a review from wmuldergov June 15, 2026 17:28
@bcgov-brwang bcgov-brwang added the ready for review The PR is ready for review label Jun 15, 2026
@bcgov-brwang bcgov-brwang marked this pull request as ready for review June 15, 2026 17:29
@wmuldergov wmuldergov added pending rework If the PR needs some re-work by the developer and removed ready for review The PR is ready for review labels Jun 16, 2026
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
@wmuldergov wmuldergov added the pending rework If the PR needs some re-work by the developer label Jun 17, 2026
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
@bcgov-brwang bcgov-brwang added ready for review The PR is ready for review and removed pending rework If the PR needs some re-work by the developer labels Jun 18, 2026
Comment thread src/frontend/src/pages/AdvisoriesListPage.js Fixed
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
Comment thread src/frontend/src/Components/shared/header/Header.js Fixed
@bcgov-brwang bcgov-brwang force-pushed the feature/DBC22-6585 branch 2 times, most recently from 2b27efa to 55ecd48 Compare June 19, 2026 21:01
@wmuldergov wmuldergov added pending rework If the PR needs some re-work by the developer and removed ready for review The PR is ready for review labels Jun 19, 2026
@bcgov-brwang bcgov-brwang added ready for review The PR is ready for review and removed pending rework If the PR needs some re-work by the developer labels Jun 22, 2026
DBC22-6585: highlighted advisory for new updates 2

DBC22-6585: highlighted bulletin for new updates

DBC22-6585: added pill on the header when advisory or bulletin updated

DBC22-6585: added publish minor updates from backend

DBC22-6585: fix bugs 1

DBC22-6585: updated details page

DBC22-6585: clear status when navigate to other pages

DBC22-6585: updated header for numbered pill

DBC22-6585: fixed bugs 1

DBC22-6585: fixed bugs 2

DBC22-6585: updated header for numbered pill test

DBC22-6585: removed unused variables

DBC22-6585: removed unused variables 1

DBC22-6585: removed unused variables 2

DBC22-6585: fixed bugs part 1

DBC22-6585: fixed bugs part 2

DBC22-6585: fixed the issue that displaying old time when last notified date is null

DBC22-6585: fixed bulletin list styling issue

DBC22-6585: test 1

DBC22-6585: fixed sync issue

DBC22-6585: test 2

DBC22-6585: recalculate unread advisories or bulletins

DBC22-6585: recalculate unread advisories or bulletins test 1

DBC22-6585: fixed github quality alerts

DBC22-6585: removed format not needed

DBC22-6585: removed bold styling for detail page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review The PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants