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 encodeWmsLayerState #28

Merged
merged 5 commits into from
Mar 21, 2024
Merged

Fix encodeWmsLayerState #28

merged 5 commits into from
Mar 21, 2024

Conversation

ismailsunni
Copy link
Collaborator

  • Set value of layers and imageFormat from params
  • Fix wrong assertions
  • Update geoblocks/print to remove the following warning
npm i --save-dev vite@latest              
npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving: @geoblocks/[email protected]
npm WARN Found: [email protected]
npm WARN node_modules/ol
npm WARN   peer ol@"7 || 8 || 9" from @geoblocks/[email protected]
npm WARN   node_modules/@geoblocks/mapfishprint
npm WARN     @geoblocks/mapfishprint@"^0.2.7" from the root project
npm WARN   2 more (@geoblocks/ol-maplibre-layer, the root project)
npm WARN 
npm WARN Could not resolve dependency:
npm WARN peer ol@"6 || 7 || 8" from @geoblocks/[email protected]
npm WARN node_modules/@geoblocks/print
npm WARN   optional @geoblocks/print@"0.7" from @geoblocks/[email protected]
npm WARN   node_modules/@geoblocks/mapfishprint
npm WARN 
npm WARN Conflicting peer dependency: [email protected]
npm WARN node_modules/ol
npm WARN   peer ol@"6 || 7 || 8" from @geoblocks/[email protected]
npm WARN   node_modules/@geoblocks/print
npm WARN     optional @geoblocks/print@"0.7" from @geoblocks/[email protected]
npm WARN     node_modules/@geoblocks/mapfishprint

Copy link
Member

@ger-benjamin ger-benjamin left a comment

Choose a reason for hiding this comment

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

Thanks

@ismailsunni ismailsunni merged commit 81e563b into main Mar 21, 2024
1 check passed
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.

3 participants