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

Method to create WPS client ExecutionResponse from status location URL #760

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

awst-baum
Copy link
Contributor

This implements the behaviour requested in issue #674:
Two (convenience) methods are added that generate an ExecutionResponse object (with execution status, percent complete, etc.) from a status location URL or stream.

To avoid code duplication, these methods are then used by ProcessExecution to parse execution responses as well.

The WPS client integration test is expanded to use the new methods.

@tfr42 tfr42 added enhancement enhancement or improvement under review PR under review by TMC member labels Aug 30, 2016
@tfr42 tfr42 added this to the 3.4.1 milestone Aug 30, 2016
@tfr42 tfr42 modified the milestones: 3.4.1, 3.5 Jun 18, 2018
@tfr42 tfr42 modified the milestones: 3.5, 3.4.3 Jul 22, 2018
@tfr42 tfr42 modified the milestones: 3.4.3, 3.4.4 Sep 28, 2018
@tfr42 tfr42 modified the milestones: 3.4.4, 3.4.5 Mar 1, 2019
@deegree-ci deegree-ci modified the milestones: 3.4.5, 3.4.6 Mar 30, 2019
@tfr42 tfr42 modified the milestones: 3.4.6, 3.4.7 May 11, 2019
@tfr42 tfr42 modified the milestones: 3.4.7, 3.4.8 Jun 14, 2019
@tfr42 tfr42 modified the milestones: 3.4.8, 3.4.9 Jul 8, 2019
@tfr42 tfr42 modified the milestones: 3.4.9, 3.4.10 Sep 6, 2019
@tfr42 tfr42 removed this from the 3.4.10 milestone Oct 25, 2019
@copierrj copierrj added the needs rebase PR is not up to date and needs rebase label Dec 6, 2019
@copierrj
Copy link
Member

copierrj commented Dec 6, 2019

Travis currently has a problem building this PR. Rebasing would likely fix this.

@tfr42 tfr42 removed the needs rebase PR is not up to date and needs rebase label Aug 15, 2021
@tfr42 tfr42 added the WPS deegree Web Processing Service label Dec 1, 2021
@tfr42 tfr42 added the needs rebase PR is not up to date and needs rebase label May 16, 2022
@tfr42 tfr42 removed the needs rebase PR is not up to date and needs rebase label Aug 10, 2022
@stephanr stephanr added the needs rebase PR is not up to date and needs rebase label May 15, 2023
@tfr42 tfr42 added stuck stuck PR and removed under review PR under review by TMC member labels Nov 30, 2023
@tfr42
Copy link
Member

tfr42 commented Nov 30, 2023

@awst-baum Thank you for your contribution. Unfortunately, it cannot be merged automatically. It cannot be considered for inclusion, before this has been fixed. Please rebase it against the target branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement enhancement or improvement needs rebase PR is not up to date and needs rebase stuck stuck PR WPS deegree Web Processing Service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants