Skip to content

Fix/map share state validation - #172

Merged
yakew7 merged 2 commits into
StudentSuite:mainfrom
techykaif:fix/map-share-state-validation
Aug 21, 2026
Merged

Fix/map share state validation#172
yakew7 merged 2 commits into
StudentSuite:mainfrom
techykaif:fix/map-share-state-validation

Conversation

@techykaif

Copy link
Copy Markdown
Contributor

What this changes

Fixes #154 by validating shared map viewport parameters before they are passed to Leaflet.

Latitude, longitude, and zoom values are now constrained to valid ranges, with regression tests covering valid boundaries and invalid values.

Type of change

  • New public place(s)
  • New or updated benefit guide
  • New or updated resource link
  • Code or fix
  • Docs

Proof (required for new public places)

Fill this in for every place added in this PR. It stays in the PR, not in the
dataset.

Field Value
Source or citation N/A
Google Maps rating (must be >= 4.0) N/A
Google Maps review count (must be >= 50) N/A
Date verified N/A
  • Each place's rating is 4.0 or higher.
  • Each place has 50 or more reviews.
  • Coordinates point to the correct entrance.
  • The JSON record stops at gmaps_link and added_by (no proof fields committed).

Checklist

  • The site hosts no copyrighted files; resource and paper changes are links only.
  • No em dashes in any copy.

@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

@techykaif is attempting to deploy a commit to the studentsuite Team on Vercel.

A member of the Team first needs to authorize it.

@yakew7
yakew7 merged commit 4f646c1 into StudentSuite:main Aug 21, 2026
3 of 4 checks passed
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.

bug: parseMapState accepts out-of-range lat/lng/zoom and feeds them straight to Leaflet

2 participants