[openstreetmap/openstreetmap-website] include iD as npm package (PR #6565)
Pablo Brasero
notifications at github.com
Wed Nov 26 13:46:40 UTC 2025
@pablobm commented on this pull request.
> @@ -22,7 +22,7 @@ document.addEventListener("DOMContentLoaded", function () {
const id = idContext
.embed(true)
- .assetPath("iD/")
+ .assetPath("@openstreetmap/id/dist/")
Looking into it, as it's indeed different for me. For the record, this is the HTML I get served with the code as in this PR, with an empty asset map:
```
<div id="id-container" data-token="9PXRfYK_vnT-6CZw4InO1XTwjnxqPnqzQkfQFvKw2kM" data-locale="en" data-asset-map="{}"></div>
```
And visibly broken UI:
<img width="753" height="181" alt="Visibly broken iD editor, with missing translation and no aerial imagery" src="https://github.com/user-attachments/assets/e922e211-4044-401f-929f-b5459ac59397" />
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6565#discussion_r2565075622
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6565/review/3511051268 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251126/6be1ab3a/attachment.htm>
More information about the rails-dev
mailing list