Skip to content

fix: show all cameras on alert maps#230

Merged
m-julio merged 3 commits into
mainfrom
fix/219-alert-map-all-cameras
Jun 17, 2026
Merged

fix: show all cameras on alert maps#230
m-julio merged 3 commits into
mainfrom
fix/219-alert-map-all-cameras

Conversation

@momocentral

Copy link
Copy Markdown
Collaborator

Summary

  • Pass the existing organization camera list into shared alert detail maps
  • Render inactive organization cameras alongside the active alert camera/cone markers
  • Include all camera positions in map bounds so the newly visible markers are in view

Fixes #219

Local checks

  • pnpm run format:check
  • pnpm lint
  • pnpm build

@vercel

vercel Bot commented May 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
new-pyro-platform Ready Ready Preview, Comment Jun 17, 2026 6:48pm

Comment thread src/components/Alerts/AlertDetails/AlertInfos/AlertInfos.tsx Outdated
Comment thread src/components/Alerts/AlertDetails/AlertInfos/AlertMap.tsx Outdated
Comment thread src/components/Alerts/AlertDetails/AlertInfos/AlertMap.tsx Outdated
@suyuen

suyuen commented May 23, 2026

Copy link
Copy Markdown
Contributor

Changes requested by @m-julio are done! see screenshots attached! :)
Screenshot 2026-05-23 at 6 09 24 PM

@theo-rodrigues-dev

Copy link
Copy Markdown
Collaborator

Thank you for your changes ! It looks good to me !

Comment thread src/components/Alerts/AlertDetails/AlertInfos/AlertMap.tsx Outdated
Comment thread src/components/Alerts/AlertDetails/AlertInfos/AlertMap.tsx Outdated
Comment thread src/pages/AlertsPage.tsx
@m-julio m-julio merged commit 1858006 into main Jun 17, 2026
3 checks passed
@m-julio m-julio deleted the fix/219-alert-map-all-cameras branch June 17, 2026 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Show all organization cameras on the Alerts page map

4 participants