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

Various fixes #570

Merged
merged 2 commits into from
Feb 3, 2020
Merged

Various fixes #570

merged 2 commits into from
Feb 3, 2020

Conversation

pelord
Copy link
Member

@pelord pelord commented Jan 31, 2020

Please check if the PR fulfills these requirements

What is the current behavior? (You can also link to an open issue here)
1- QGis server do not allow lowercase wms or wfs service.
2- For layer with fields aliases, if the field was empty, the field was not shown in the results.

What is the new behavior?
1- set uppercase
2- keep all the fields

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

@pelord pelord changed the title NOT READY -- Various Various fixes Jan 31, 2020
@pelord pelord marked this pull request as ready for review January 31, 2020 18:12
@pelord pelord requested a review from mbarbeau January 31, 2020 18:12
@mbarbeau mbarbeau merged commit 37e2276 into master Feb 3, 2020
@mbarbeau mbarbeau deleted the various branch February 3, 2020 12:21
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.

2 participants