Skip to content

Map test with UMD build#395

Draft
alexluckett wants to merge 3 commits into
maplibre-peer-dependencyfrom
demo/map-test
Draft

Map test with UMD build#395
alexluckett wants to merge 3 commits into
maplibre-peer-dependencyfrom
demo/map-test

Conversation

@alexluckett

Copy link
Copy Markdown
Contributor

No description provided.

Imports the accessibility test site from defra-design/map-test under
demo/, pointing @defra/interactive-map at the local repo (file:../..)
and flattening the old plugins/beta and providers/beta plugin-asset
paths to match the monorepo structure.

All 11 pages verified in headless Chromium against the local UMD
builds: defra.* globals resolve and the map boots to a live canvas
on every page.
Replaces the file:../.. prototype-kit plugin wiring with the exact
consumer setup: scripts/install-local-packages.sh packs core plus the
7 provider/plugin packages the demo uses, npm-installs the tarballs
(npm resolves maplibre-gl via the new required peer deps), and copies
each package's dist into app/assets/vendor. Views now load the UMD
bundles with standard <script src> tags instead of /plugin-assets/.

All 11 pages re-verified in headless Chromium: defra.* globals
resolve and the map boots to a live canvas.
@alexluckett alexluckett marked this pull request as draft July 9, 2026 10:47
@alexluckett alexluckett changed the title Demo/map test Map test with UMD build Jul 9, 2026
@sonarqubecloud

sonarqubecloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

Gives every UMD-shipping workspace package a govuk-prototype-kit.config.json
(exposing /dist/umd and /dist/css where present) so prototype kit apps that
install the split packages get their bundles served natively under
/plugin-assets/<package>/dist/... — matching the install-only-what-you-use
model. esri and draw-es are skipped (ESM-only, no UMD build).

map-test now loads each package via its own plugin-assets path, dropping
the vendor-copy step. All 11 pages re-verified in headless Chromium.
@sonarqubecloud

sonarqubecloud Bot commented Jul 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.

1 participant