Skip to content

DBC22-6162: reset map center based on root camera location - #1289

Closed
bcgov-brwang wants to merge 3 commits into
mainfrom
bugfix/DBC22-6162
Closed

DBC22-6162: reset map center based on root camera location#1289
bcgov-brwang wants to merge 3 commits into
mainfrom
bugfix/DBC22-6162

Conversation

@bcgov-brwang

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)

  1. Go to camera details page, resize browser window small until nearby tab shows up
  2. Verify if the mini map can be shown correctly instead of showing a wrong map center

🔍 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 wmuldergov April 9, 2026 19:45
@bcgov-brwang
bcgov-brwang marked this pull request as ready for review April 9, 2026 19:46
@wmuldergov

Copy link
Copy Markdown
Collaborator

As discussed, the zoom level on the Nearby Area Map seems to lock in at 200m (zoom level 15). It should match what is in prod where it stays the same for both versions of the map.

@sonarqubecloud

sonarqubecloud Bot commented Apr 9, 2026

Copy link
Copy Markdown

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.

2 participants