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

Decision Services & multiple DRDs: Adding a Decision Service to a DRD. #888

Closed
Tracked by #625
tiagobento opened this issue Feb 1, 2024 · 1 comment · Fixed by apache/incubator-kie-tools#2181
Assignees
Labels
area:dmn Related to DMN area:tools Issues affecting Apache KIE tooling projects type:enhancement Something that already exists needs to be improved

Comments

@tiagobento
Copy link

tiagobento commented Feb 1, 2024

On the DMN Editor, when a Decision Service is added to a DRD, all of its contained Decisions need to be moved inside it or added to the DRD, so that the internal depiction of that Decision Service is exactly the same as all other DRDs, or automatically layout'ed if none is found. However, if one of its contained Decisions it also contained by other Decision Services present on that DRD, and one of them is in expanded form, the Decision Service is added as collapsed, and no Decisions are added or moved.

EDIT: When the collapsed case happens, we also need to check for any Decisions that are contained by the Decision Service that is being added as collapsed. Those Decisions need to be removed from the DRD, as they're "swallowed" by the Decision Service.

@tiagobento tiagobento self-assigned this Feb 2, 2024
@tiagobento tiagobento added type:enhancement Something that already exists needs to be improved area:dmn Related to DMN area:tools Issues affecting Apache KIE tooling projects labels Feb 2, 2024
@tiagobento tiagobento changed the title Decision Services & multiple DRDs: Adding a Decision Service to a DRD Decision Services & multiple DRDs: Adding a Decision Service to a DRD. Feb 2, 2024
@tiagobento tiagobento moved this from 📋 Backlog to ⏳ In Progress in 🦉 KIE Podling Board Feb 6, 2024
@tiagobento
Copy link
Author

@ljmotta Found a case we forgot... added to the description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dmn Related to DMN area:tools Issues affecting Apache KIE tooling projects type:enhancement Something that already exists needs to be improved
Projects
Archived in project
1 participant