Skip to content

Fix SAT centre city slug collisions - #329

Merged
yakew7 merged 1 commit into
StudentSuite:mainfrom
jay23606:pr-go/284-sat-centres-in-the-us-and-canada-filed-u
Nov 1, 2026
Merged

yakew7 merged 1 commit into
StudentSuite:mainfrom
jay23606:pr-go/284-sat-centres-in-the-us-and-canada-filed-u

Conversation

@jay23606

@jay23606 jay23606 commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Bug

SAT centres in North American cities shared slugs with unrelated cities, mixing their city pages and map bounds. Two entries also had coordinates that did not match their verified addresses.

Change

Qualified the affected city slugs by state or province, kept the Canary Islands school under arona, corrected the Reading, Pennsylvania and Southport, Queensland coordinates and map links, and documented the same-name slug rule.

Testing

  • npx vitest run src/lib/places.test.ts (fails before the data change and passes after)
  • npm run validate

Regression test: fails on base (/expected 'aberdeen' to be 'aberdeen_maryland'/), passes on fix (npx vitest run src/lib/places.test.ts)

Fixes #284

@vercel

vercel Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

@jay23606 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 b1d5146 into StudentSuite:main Nov 1, 2026
4 of 5 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.

SAT centres in the US and Canada filed under same-named UK and EU city slugs

2 participants