Skip to content

Remove ID required for documentation and loops#8

Open
vehikl-jacare wants to merge 4 commits intoProcessMaker:developfrom
vehikl-jacare:remove-id-required-for-documentation-and-loops
Open

Remove ID required for documentation and loops#8
vehikl-jacare wants to merge 4 commits intoProcessMaker:developfrom
vehikl-jacare:remove-id-required-for-documentation-and-loops

Conversation

@vehikl-jacare
Copy link
Copy Markdown

Fixes Modeler 1255.

This adds two conditions for early return to the ID required validation:

  • if the node is a documentation node
  • if the node is a loop characteristic node.

Two test fixtures have been added and the test suite updated.

vehikl-jacare and others added 4 commits January 4, 2021 10:47
Co-Authored-By: Justin Struk <JStruk@users.noreply.github.com>
Co-Authored-By: Robert Komaromi <chipit24@users.noreply.github.com>
Co-Authored-By: Gerrie Swart <johnblackspear@users.noreply.github.com>
Co-Authored-By: Justin Struk <JStruk@users.noreply.github.com>
Co-Authored-By: Robert Komaromi <chipit24@users.noreply.github.com>
Co-Authored-By: Gerrie Swart <johnblackspear@users.noreply.github.com>
Co-Authored-By: Justin Struk <JStruk@users.noreply.github.com>
Co-Authored-By: Robert Komaromi <chipit24@users.noreply.github.com>
Co-Authored-By: Gerrie Swart <johnblackspear@users.noreply.github.com>
Co-Authored-By: Justin Struk <JStruk@users.noreply.github.com>
Co-Authored-By: Robert Komaromi <chipit24@users.noreply.github.com>
Co-Authored-By: Gerrie Swart <johnblackspear@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Element is missing ID" error popping up in the BPMN issues part, but no element is being highlighted

1 participant