Skip to content

DBC22-6850: fixed issue that advisory link cannot be navigated to det… - #1391

Merged
ray-oxd merged 1 commit into
mainfrom
bugfix/DBC22-6850
Jul 15, 2026
Merged

DBC22-6850: fixed issue that advisory link cannot be navigated to det…#1391
ray-oxd merged 1 commit into
mainfrom
bugfix/DBC22-6850

Conversation

@bcgov-brwang

Copy link
Copy Markdown
Collaborator

…ails page

📝 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)

  1. Deploy to dev.
  2. On the either map, find an advisory and click it to open the sidepanel.
  3. Click on either of the links: Advisory name and View Details.
  4. Verify it can be navigated to advisory detail page.

🔍 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

@bcgov-brwang
bcgov-brwang requested a review from ray-oxd July 8, 2026 14:15
@bcgov-brwang bcgov-brwang added the ready for review The PR is ready for review label Jul 8, 2026
@bcgov-brwang
bcgov-brwang marked this pull request as ready for review July 8, 2026 14:15
@ray-oxd
ray-oxd merged commit cf5c468 into main Jul 15, 2026
5 of 6 checks passed
@ray-oxd
ray-oxd deleted the bugfix/DBC22-6850 branch July 15, 2026 00:05
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.

2 participants