Skip to content

fix: transform mixed esm modules#1910

Merged
fayi-da merged 1 commit into
mainfrom
fayi/transform-esm-portfolio
May 29, 2026
Merged

fix: transform mixed esm modules#1910
fayi-da merged 1 commit into
mainfrom
fayi/transform-esm-portfolio

Conversation

@fayi-da
Copy link
Copy Markdown
Contributor

@fayi-da fayi-da commented May 29, 2026

No description provided.

Signed-off-by: Fayi Femi-Balogun <fayimora.femibalogun@digitalasset.com>
@fayi-da fayi-da marked this pull request as ready for review May 29, 2026 14:28
@fayi-da fayi-da requested a review from a team as a code owner May 29, 2026 14:28
@fayi-da fayi-da requested a review from alexmatson-da May 29, 2026 14:28
Copy link
Copy Markdown
Contributor

@alexmatson-da alexmatson-da left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you havent yet, maybe worth testing the bundle by running some local web server (like python -m http.server) from the dist/ directory. If that works, then i'm reasonably confident the image should work

@fayi-da
Copy link
Copy Markdown
Contributor Author

fayi-da commented May 29, 2026

if you havent yet, maybe worth testing the bundle by running some local web server (like python -m http.server) from the dist/ directory. If that works, then i'm reasonably confident the image should work

@alexmatson-da I actually went further and built a docker image using a similar Dockerfile in the infra repo. Ran it on port 3333, added 3333 for cors to WG and tested a simple flow. All worked well

@fayi-da fayi-da merged commit 3c9077d into main May 29, 2026
27 checks passed
@fayi-da fayi-da deleted the fayi/transform-esm-portfolio branch May 29, 2026 15:24
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.

2 participants