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

Keep in the model the information of which edges are reverted #31

Merged
merged 1 commit into from
May 6, 2020

Conversation

baptistemesta
Copy link
Contributor

This is required to be able to position shape

Also removed generated Ids from shape, there is no need to have generated ids as BPMN file already have id.

public static Edge edge(String from, String to) {
return new Edge(generateRandomId(), from, to, false);
}
public static Edge revertedEdge(Edge original) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

way to much edge builders...

This is required to be able to position shape

Also removed generated Ids from shape, there is no need to have generated ids as BPMN file already have id.
@baptistemesta baptistemesta merged commit 7ed5492 into master May 6, 2020
@baptistemesta baptistemesta deleted the reverted_edges branch May 6, 2020 14:25
@tbouffard tbouffard added the chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants