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

Upgrade/id #6548

Closed
wants to merge 6 commits into from
Closed

Upgrade/id #6548

wants to merge 6 commits into from

Conversation

emi420
Copy link
Collaborator

@emi420 emi420 commented Sep 2, 2024

What type of PR is this?

  • ❓ Other (package upgrade)

Related Issue

Fixes #5780

Describe this PR

A new version (2.2.29) of the iD Editor was published as a NPM package on hotosm/id

This PR upgrades the package and also adds a configuration line for Webpack to ignore and not parse the iD.min.js file, as it was raising some warnings:

Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Critical dependency: the request of a dependency is an expression

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.

WARNING in ./node_modules/@hotosm/id/dist/iD.min.js 14:40-47
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

WARNING in ./node_modules/@hotosm/id/dist/iD.min.js 17:37-44
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

WARNING in ./node_modules/@hotosm/id/dist/iD.min.js 40022:67-76
Critical dependency: the request of a dependency is an expression

It should be ok as this is a packed and minified file.

Copy static files and imagery patch

I've also divided the commands in package.json for copying static files for both iD and Rapid and removed the patch for the iD imagery as isn't being used anymore.

@github-actions github-actions bot added dependencies Pull requests that update a dependency file scope: frontend labels Sep 2, 2024
Copy link

sonarcloud bot commented Sep 2, 2024

@emi420
Copy link
Collaborator Author

emi420 commented Sep 3, 2024

I close this PR and will create a new one for iD v2.30.2

@emi420 emi420 closed this Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file scope: frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

iD editor version upgrade
3 participants