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

JBPM-9607 Allow WS work item handler to support wrapped parameter sty… #1865

Merged
merged 1 commit into from Feb 16, 2021

Conversation

cristianonicolai
Copy link
Contributor

…le in remote call

Thank you for submitting this pull request

JIRA:

https://issues.redhat.com/browse/JBPM-9607

This PR adds a new parameter to the WebService WIH. In order to test the new behaviour, please create a new WIH call using `Wrapped` : 'True` in the data assignments. Also, send a wrapping class object as the parameter.

@cristianonicolai
Copy link
Contributor Author

jenkins please retest this

Copy link
Member

@MarianMacik MarianMacik left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@cristianonicolai
Copy link
Contributor Author

jenkins please retest this

@elguardian
Copy link
Member

there is one test failure not related regarding human task. should be ignored for merging.

@sonarcloud
Copy link

sonarcloud bot commented Feb 15, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@cristianonicolai
Copy link
Contributor Author

Jenkins run fdb

@cristianonicolai
Copy link
Contributor Author

merging as human task test error is not related.

@cristianonicolai cristianonicolai merged commit f9dbb65 into kiegroup:master Feb 16, 2021
@cristianonicolai cristianonicolai deleted the JBPM-9590 branch February 16, 2021 05:48
fjtirado pushed a commit to fjtirado/jbpm that referenced this pull request Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants