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

Add signal events support in the bpmn model api #10530

Closed
skayliu opened this issue Sep 28, 2022 · 5 comments · Fixed by #10589
Closed

Add signal events support in the bpmn model api #10530

skayliu opened this issue Sep 28, 2022 · 5 comments · Fixed by #10589
Labels
hacktoberfest Marks an issue as a candidate to be a Hacktoberfest contribution kind/feature Categorizes an issue or PR as a feature, i.e. new behavior version:8.2.0-alpha1 Marks an issue as being completely or in parts released in 8.2.0-alpha1 version:8.2.0 Marks an issue as being completely or in parts released in 8.2.0

Comments

@skayliu
Copy link
Contributor

skayliu commented Sep 28, 2022

Is your feature request related to a problem? Please describe.

As a user, i'd like to there is apis support for signal event modelling. And finally we can run signal events in C8.

Describe the solution you'd like

First thing first, we should add signal event support in the bpmn model api of C8

Describe alternatives you've considered

None

Additional context

Align the capabilities with C7.
Events
Signal Events
BPMN2.0

@skayliu skayliu added the kind/feature Categorizes an issue or PR as a feature, i.e. new behavior label Sep 28, 2022
@saig0 saig0 added the hacktoberfest Marks an issue as a candidate to be a Hacktoberfest contribution label Sep 29, 2022
@skayliu
Copy link
Contributor Author

skayliu commented Sep 29, 2022

Hi,@saig0, I'd love to make a PR for this, may i ?

@saig0
Copy link
Member

saig0 commented Sep 29, 2022

@skayliu awesome. 🎉 I'm looking forward to your PR. 🚀

If you want to participate in the Hacktoberfest then you should wait until the 1st of October.


Please be aware that the implementation of the execution logic of signal events is not defined yet. Signal events may end up similar to message events but distributed to all partitions. I could imagine that a signal event has only a name but no correlation key or a time-to-live (TTL).

So, the implementation effort could be very high and include a lot of communication to define the expected behavior/feature first.

@skayliu
Copy link
Contributor Author

skayliu commented Sep 29, 2022

Yes, I totally understand, It's more complex then message events, so i want to support the bpmn model api first.

@skayliu
Copy link
Contributor Author

skayliu commented Sep 29, 2022

@saig0, Also, the bpmn model api for conditional events is almost done.

@saig0
Copy link
Member

saig0 commented Sep 30, 2022

@skayliu great. 👍

We just need to make sure to not expose or allow features in the Model API that are not supported yet (or in the near future).

@korthout korthout added the version:8.2.0-alpha1 Marks an issue as being completely or in parts released in 8.2.0-alpha1 label Nov 1, 2022
@npepinpe npepinpe added the version:8.2.0 Marks an issue as being completely or in parts released in 8.2.0 label Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest Marks an issue as a candidate to be a Hacktoberfest contribution kind/feature Categorizes an issue or PR as a feature, i.e. new behavior version:8.2.0-alpha1 Marks an issue as being completely or in parts released in 8.2.0-alpha1 version:8.2.0 Marks an issue as being completely or in parts released in 8.2.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants