Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix es-build angular 16 'Cannot assign to import "baseApiUrl"' #408

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

harellevy
Copy link
Contributor

@harellevy harellevy commented Oct 17, 2023

fixing es-build angular 16 build with error

Cannot assign to import "baseApiUrl"

    node_modules/ngx-mapbox-gl/fesm2022/ngx-mapbox-gl.mjs:34:25:
      34 │                 MapboxGl.baseApiUrl = options.customMapboxApiUrl;
         ╵                          ~~~~~~~~~~

  Imports are immutable in JavaScript. To modify the value of this import

related to evanw/esbuild#891, maplibre/maplibre-gl-js#1550

fixes:
#403
#407

fixing es-build angular 16 build with error
```
Cannot assign to import "baseApiUrl"

    node_modules/ngx-mapbox-gl/fesm2022/ngx-mapbox-gl.mjs:34:25:
      34 │                 MapboxGl.baseApiUrl = options.customMapboxApiUrl;
         ╵                          ~~~~~~~~~~

  Imports are immutable in JavaScript. To modify the value of this import
```
related to evanw/esbuild#891,



fixes:
Wykks#403
Wykks#407
@LcsGa
Copy link

LcsGa commented Oct 30, 2023

Hey, do you have any news for this PR ? I really need this fix 😬 @Wykks

@Wykks
Copy link
Owner

Wykks commented Nov 1, 2023

Hi, I should have looked into this before, LTGM :)

Copy link

nx-cloud bot commented Nov 1, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 0c38932. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


🟥 Failed Commands
nx affected --target=build --parallel=3
✅ Successfully ran 3 targets

Sent with 💌 from NxCloud.

@Wykks Wykks merged commit 7942ca3 into Wykks:main Nov 1, 2023
4 of 5 checks passed
@LcsGa
Copy link

LcsGa commented Dec 12, 2023

Hi ! Will there be a new release soon? I still can't use esbuild because of this package...

@BergkampHUN
Copy link
Contributor

@Wykks can you release this fix? The pipeline failed so it was never released.

@atocqueville
Copy link

@Wykks Could you release this fix please ? 🙏

@bajere
Copy link

bajere commented Jun 28, 2024

@Wykks is this dead? We can update the package directly, but it will be flaky for deployments. Thanks for the work too!

denisyilmaz pushed a commit to denisyilmaz/ngx-mapbox-gl that referenced this pull request Sep 10, 2024
fixing es-build angular 16 build with error
```
Cannot assign to import "baseApiUrl"

    node_modules/ngx-mapbox-gl/fesm2022/ngx-mapbox-gl.mjs:34:25:
      34 │                 MapboxGl.baseApiUrl = options.customMapboxApiUrl;
         ╵                          ~~~~~~~~~~

  Imports are immutable in JavaScript. To modify the value of this import
```
related to evanw/esbuild#891,



fixes:
Wykks#403
Wykks#407
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.

7 participants