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-9611] Command resource returns HTTP OK response status code even for failure scenarios #2396

Merged
merged 1 commit into from Feb 24, 2021

Conversation

fjtirado
Copy link
Contributor

API and test changed to return different HTTP codes depeding on service result.

JIRA:

link

Copy link
Contributor

@afalhambra afalhambra left a comment

Choose a reason for hiding this comment

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

only a minor comment. Otherwise, looks good to me.

@fjtirado fjtirado force-pushed the JBPM-9611 branch 2 times, most recently from 131966b to f9905c7 Compare February 16, 2021 15:43
Copy link
Contributor

@afalhambra afalhambra left a comment

Choose a reason for hiding this comment

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

Great work @fjtirado

Copy link
Member

@gmunozfe gmunozfe 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, just a typo that makes compilation fail

@sonarcloud
Copy link

sonarcloud bot commented Feb 23, 2021

Copy link
Member

@elguardian elguardian left a comment

Choose a reason for hiding this comment

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

backward compatiblity + Test Case
LGTM

@elguardian elguardian merged commit 215a588 into kiegroup:master Feb 24, 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
4 participants