Feature/XIVY-13085 add business notification demo #514
Conversation
weissreto
force-pushed
the
feature/XIVY-13085_add_notification_demo
branch
from
85b942b to
50efafc
Compare
|
|
||
| @Test | ||
| void runProcess(BpmClient bpmClient) | ||
| { |
There was a problem hiding this comment.
Yes, the whole demo is formatted this way.
There was a problem hiding this comment.
Can change in a a different PR. But then all demos.
There was a problem hiding this comment.
Have changed this class to legazy formatting only. Will convert every demo to new formatting in a new PR after my holidays.
| "config" : { | ||
| "data" : "workflow.order.OrderData" | ||
| }, | ||
| "elements" : [ { |
There was a problem hiding this comment.
I'd add an 'annotation' on this process, explaining the purpose of this demo. ... as you see e.g. in procurementRequest
| "type" : "DialogCall", | ||
| "name" : "Input Order", | ||
| "config" : { | ||
| "dialog" : "workflow.order.InputOrder:start()", |
There was a problem hiding this comment.
the initial order form is empty ... which is realistic, but bad for demonstrations.
I'd prefil the order form with 'John Doe' and the Ferrari you are also using in tests.
So I can demonstrate the feature without touching the keyboard...
weissreto
force-pushed
the
feature/XIVY-13085_add_notification_demo
branch
2 times, most recently
from
381ec31 to
21fafce
Compare
| "name" : [ | ||
| "**Business Notification Demo**", | ||
| "", | ||
| "This demo shows you how you can send business notifications in your process.", |
There was a problem hiding this comment.
double 'you' --> This demo shows how you can ...
| "**Business Notification Demo**", | ||
| "", | ||
| "This demo shows you how you can send business notifications in your process.", | ||
| "Starts with the simplest way first, then shows how you can use process data in your messages, use multi-lingual messages, and finally how to use templates." |
Add a new demo process 'OrderProcess' that demonstrates the different use cases how the BusinessNotification API can be used. Add a test for the new process
weissreto
force-pushed
the
feature/XIVY-13085_add_notification_demo
branch
from
21fafce to
9dffa34
Compare
No description provided.