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

format problem in WMS WMTS #175

Open
thanatas opened this issue Aug 14, 2023 · 4 comments
Open

format problem in WMS WMTS #175

thanatas opened this issue Aug 14, 2023 · 4 comments

Comments

@thanatas
Copy link

I think I got a bug;

at version @1.5.3 WMS and WMTS layers do not accept format as parameter .

In the code (RLayerWMS and RLayerWMTS) createSource() -> optionsFromCapabilities, only layer parameter is passed. Adding, format is a option parameter.

                const caps = this.parser.read(text);
                this.options = optionsFromCapabilities(caps, {
                    layer: this.props.layer
                });

In geoserver, if you enable multiple tile caching types, optionsFromCapabilities function select first one from server which is the type of json and it is not compatible with wms and wmts.

Could you check that OR WHAT am I doing wrong.

Thanks.

@mmomtchev
Copy link
Owner

Yes, I confirm, if you add it, I will merge the PR

@thanatas
Copy link
Author

thanatas commented Aug 21, 2023

Hi again, I tried to fix and tested on local.

Unfortunately, without changing the package.json build scripts I could not build the project. If I may so, I will send a merge request.

I got the following error; I could not find what is the problem.

image

I am using windows 11, I tried nodejs 18,19,20. What is required to do so?

Thanks

@mmomtchev
Copy link
Owner

This is a bundler problem, what makes you think that the problem is not what it says - that iife is not supported as an output format for code-splitting builds?

mmomtchev pushed a commit that referenced this issue Sep 2, 2023
Fixes: format problem in WMS WMTS #175

Error: UMD and IIFE output formats are not supported for code-splitting builds.
@mmomtchev
Copy link
Owner

This is a problem when rebuilding the library for publishing on Windows, you can safely ignore it

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

No branches or pull requests

2 participants