Skip to content

Feature/migrate to real data#106

Closed
james-cross wants to merge 6 commits intoStreetSupport:stagingfrom
james-cross:feature/migrate-to-real-data
Closed

Feature/migrate to real data#106
james-cross wants to merge 6 commits intoStreetSupport:stagingfrom
james-cross:feature/migrate-to-real-data

Conversation

@james-cross
Copy link
Collaborator

This PR completes the migration from local JSON mocks to fully dynamic, database-driven data for the Find Help flow and Organisation entry pages. It also includes aligned unit tests, updated E2E tests, and resolves all type/lint issues related to the new normalised data shape.

Included in this PR
Find Help:

Switched from static service-providers.json to live API routes.

Uses real database categories, client groups, services, and location-aware results.

Updated search logic to correctly order, filter, and group services.

Organisation Pages:

Implemented dynamic [slug] route fetching real organisation data, addresses, and provided services.

Added proper normalisation: all keys are consistently lowercased (name, addresses, etc.).

Built OrganisationShell with modular subcomponents: Overview, Locations, ServicesAccordion, ContactBlock, Footer.

Connected real grouped services, addresses with maps, and social share links.

Testing:

Updated unit tests to mock realistic API responses.

Fixed component tests for OrganisationOverview, OrganisationLocations, OrganisationServicesAccordion, and ContactBlock.

Refreshed Playwright E2E tests to verify real org pages, map rendering, accordion toggle, and share/disclaimer behaviour.

All tests pass and linting is clean.

Notes
The API shape is now finalised for both Find Help and Organisation details.

This ensures future styling and features are built on real, production-like data.

Minor tweaks may be required when plugging in final location coordinates or address formats.

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.

1 participant