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

refactor/scenario6-10 #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

refactor/scenario6-10 #15

wants to merge 3 commits into from

Conversation

MartinWitt
Copy link
Contributor

No description provided.

@MartinWitt MartinWitt requested a review from layornos July 5, 2021 17:18
@MartinWitt MartinWitt changed the title refactor/scenario6 refactor/scenario6-10 Jul 5, 2021
Copy link
Contributor

@layornos layornos left a comment

Choose a reason for hiding this comment

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

Viel stuff, das müssen wir morgen besprechen

@@ -59,7 +63,19 @@
import org.camunda.bpm.model.bpmn.instance.paradigm.looping.MultiInstanceLoopCharacteristics;
import org.camunda.bpm.model.bpmn.instance.paradigm.messaging.Message;
import org.camunda.bpm.model.bpmn.instance.paradigm.services.Error;
import org.camunda.bpm.model.bpmn.instance.domain.processes.Process;
import org.camunda.bpm.model.bpmn.instance.paradigm.activities.Activity;
Copy link
Contributor

Choose a reason for hiding this comment

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

Ist eine Trennung von Paradigm und Domain hier nicht möglich?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

das ist in before?

@@ -1590,6 +1598,7 @@ public void testIntermediateSignalThrowEventWithPayload() {
assertThat(camundaInParams.get(0).getCamundaVariables()).isEqualTo("all");
}

@Domain
Copy link
Contributor

Choose a reason for hiding this comment

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

Das soll signalisieren, dass es sich um ein Test der Domain Layer handelt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ich glaube das war einer der gescheiterten versuche und kann ignoriert werden

@MartinWitt
Copy link
Contributor Author

Ich kann dir die PR auch splitten, wenn es hilft

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.

2 participants