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

Wps execute accept language #74

Merged
merged 6 commits into from
Feb 10, 2020
Merged

Conversation

davidcaron
Copy link
Contributor

@davidcaron davidcaron commented Feb 7, 2020

Support the Accept-Language header when executing a WPS process, match it to the best-supported language (according to the GetCapabilities call) and forward it to owslib.wps.WebProcessingService.

Fixes #73

Requires geopython/OWSLib#655 and geopython/OWSLib#654 to be released in the next owslib version. This PR shouldn't break anything in the meantime, it just doesn't do anything.

@github-actions github-actions bot added feature/db Related to database or datatype manipulation. ci/tests Tests of the package and features process/wps3 Issue related to WPS 3.x (REST-JSON) processes support labels Feb 7, 2020
@davidcaron
Copy link
Contributor Author

I think secret scan was already failing, and the Codacy errors are not important.

Copy link
Collaborator

@fmigneault fmigneault left a comment

Choose a reason for hiding this comment

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

good to go after small doc fix

weaver/wps_restapi/processes/processes.py Show resolved Hide resolved
@fmigneault
Copy link
Collaborator

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 2
           

See the complete overview on Codacy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/tests Tests of the package and features feature/db Related to database or datatype manipulation. process/wps3 Issue related to WPS 3.x (REST-JSON) processes support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support multiple languages for wps 1 providers
2 participants