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

Consider allowing integration handlers to consume events. #89

Open
jmalloc opened this issue Jul 2, 2019 · 4 comments
Open

Consider allowing integration handlers to consume events. #89

jmalloc opened this issue Jul 2, 2019 · 4 comments
Labels
needs adr The changes discussed require an ADR undecided Further discussion is required to how (or whether) to proceed

Comments

@jmalloc
Copy link
Member

jmalloc commented Jul 2, 2019

See also #90.

The idea here is that integrations with other systems may need to respond to events that occur, without necessarily involving a business process.

One hypothetical example would be sending a notification to online users about completion of a transaction.

@jmalloc jmalloc added the needs adr The changes discussed require an ADR label Jul 2, 2019
@jmalloc jmalloc changed the title Allow integration handlers to consume events and produce commands. Consider allowing integration handlers to consume events. Jul 2, 2019
@jmalloc
Copy link
Member Author

jmalloc commented Jul 2, 2019

@danilvpetrov Do you have any use-cases from Hive where this might apply?

@danilvpetrov
Copy link
Member

danilvpetrov commented Jul 2, 2019

Not at the moment, even all those events that are going to the processes should be probably kept this way as these processes guard their state and fire commands to the integration handlers accordingly. However, I can envisage the situation where some standalone integration handlers would need to listen to the events published by aggregates (even processes).

@jmalloc
Copy link
Member Author

jmalloc commented Oct 23, 2019

I'm closing this due to a lack of necessity. We can reopen it if we find legitimate use cases.

The example given above:

One hypothetical example would be sending a notification to online users about completion of a transaction.

Could be reasonably modelled as a projection anyway.

@jmalloc jmalloc closed this as completed Oct 23, 2019
@jmalloc
Copy link
Member Author

jmalloc commented Nov 9, 2021

We've discovered a use-case for this at cwx wherein we need to push an ephemeral notification to a connected client.

/cc @mrubiosan

@jmalloc jmalloc reopened this Nov 9, 2021
@jmalloc jmalloc added the undecided Further discussion is required to how (or whether) to proceed label Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs adr The changes discussed require an ADR undecided Further discussion is required to how (or whether) to proceed
Projects
Development

No branches or pull requests

2 participants