maplibre-gl-js icon indicating copy to clipboard operation
maplibre-gl-js copied to clipboard

Published 20 hours ago • verified publisher icon maplibre

standard-minifier-js of Meteor.js remove Promise

Open rtalon83 opened this issue 10 months ago • 4 comments

A Meteor.js v2.5.8 web application using MapLibre 4.1.2 to show a simple map in a dashboard. Simple mongo links and show differents layers. In development mode all is ok. But, When is production mode the app not run.

dist/maplibre-gl.js has this functions:

image

Promise is changed for arbitrary variable/string, in this case 'n'

image

maplibre-gl-js version: 4.1.2

browser: Chrome 123.0.6312.123

Steps to Trigger Behavior

  1. Create a new package from Meteor.js adding maplibre-gl library
  2. Set this package in packages folder.
  3. Build in production mode
  4. Map is visible but the communications are broken.

Consider using the non-minified version for this demo to create better error messages. E.g. <script src='https://unpkg.com/[email protected]/dist/maplibre-gl-dev.js'></script><link href='https://unpkg.com/[email protected]/dist/maplibre-gl.css' rel='stylesheet' />

See https://stackoverflow.com/help/mcve for guidelines on creating an effective example. -->

Expected Behavior

Actual Behavior

rtalon83 avatar Apr 15 '24 12:04 rtalon83

It is not clear what the problem is and how to reproduce it. This looks like a bad behavior of a minifier that didn't do its job well, I'm not sure how this is related to maplibre...? I see the word promise in both: https://unpkg.com/browse/[email protected]/dist/maplibre-gl-dev.js and https://unpkg.com/browse/[email protected]/dist/maplibre-gl.js

HarelM avatar Apr 15 '24 13:04 HarelM

Yes, is a bad behaviour of a minifier of Meteor.js, I don't know why... I write this issue because perhaps someone else has experienced the same thing. Thanks.

rtalon83 avatar Apr 15 '24 13:04 rtalon83

@rtalon83 , is this still a problem with Meteor.js v3? It has e.g. replaced the quite problematic meteor Fibers with native async/await support, so I think there's a good chance it might solve the issue.

birkskyum avatar Aug 05 '24 13:08 birkskyum

I have not tested my app with meteor.js v3, yet. But I resolved my problem changing the strategy (no package, code inside the main app). My next test will be check v3

El lun., 5 ago. 2024 15:51, Birk Skyum @.***> escribió:

@rtalon83 https://github.com/rtalon83 , is this still a problem with Meteor.js v3? It has e.g. replaced the fibers with native async/await support, so it could make a difference.

— Reply to this email directly, view it on GitHub https://github.com/maplibre/maplibre-gl-js/issues/4001#issuecomment-2269130736, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC3S7DH3JXJMCPFR257ZAMTZP57MRAVCNFSM6AAAAABGHG4VF6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRZGEZTANZTGY . You are receiving this because you were mentioned.Message ID: @.***>

rtalon83 avatar Aug 05 '24 17:08 rtalon83