Skip to content

Merge upstream v0.0.19#5

Open
saj-derilinx wants to merge 58 commits into
masterfrom
upstream_test
Open

Merge upstream v0.0.19#5
saj-derilinx wants to merge 58 commits into
masterfrom
upstream_test

Conversation

@saj-derilinx
Copy link
Copy Markdown

No description provided.

smotornyuk and others added 24 commits April 9, 2021 01:58
Support CKAN 2.9 WebAssets....
As long as "shp" or "shapefile" is present in the resource name
[ckan#87] Add on_same_domain check to WMTS and Shapefile viewer
This causes problems with sites that have wildcard CORS set up:
"The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'."

So I was just wondering if there was an overt reason for setting this up this way, or if we can change it, with this provocative commit/PR :-)
I am not familiar with ESRI's API, but this did the trick in dealing with the dataset we are looking at, and shouldn't break anything else
@saj-derilinx saj-derilinx requested a review from EricSoroos April 21, 2022 13:51
@saj-derilinx saj-derilinx mentioned this pull request Apr 21, 2022
@EricSoroos
Copy link
Copy Markdown

I'm not sure that I actually want to merge this one to master, but more generally make this the main branch.

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.