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

RHPAM-1734: Stunner - Configure rules for avoiding nesting event sub-process into other sub-processes #2298

Merged
merged 1 commit into from Nov 22, 2018

Conversation

wmedvede
Copy link
Member

No description provided.

@wmedvede
Copy link
Member Author

Hello @romartin @hasys , this PR includes the rules configuration we talked about in the "[Stunner] Blockers status for 7.2 CR2" emails thread.
Since it was decided to not include https://issues.jboss.org/browse/JBPM-7852 in the release, with this configuration we can at least prevent event subprocesses to be nested into other subprocesses.

I've tested it locally and worked fine during my checks, of course It doesn't solve other issues, but at least event subprocesses can't be added into other sub-processes and thus minimize the chance to run into the other issues.

Would you mind take look and test?

Thanks!

@romartin romartin requested a review from hasys November 22, 2018 00:00
@romartin
Copy link
Contributor

Jenkins execute full downstream build

@wmedvede
Copy link
Member Author

Hello @hasys , it seems like we have the two greens. Would you mind take a look please?
Thanks!

Copy link
Member

@hasys hasys left a comment

Choose a reason for hiding this comment

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

Hi @wmedvede,

looks and works good, I can add any type of sub-process but Event sub-process into another any sub-process in any supported browser.

Thank you!

@wmedvede
Copy link
Member Author

Hi @romartin, I think this is ready for merge.

@romartin romartin merged commit 0bc0800 into kiegroup:7.14.x Nov 22, 2018
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