Skip to content

Commit

Permalink
fixup! docs: adr for enabling producing to event bus via settings
Browse files Browse the repository at this point in the history
Co-authored-by: Robert Raposa <[email protected]>
  • Loading branch information
navinkarkera and robrap committed Jun 6, 2023
1 parent 636ed8c commit fd624f9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions docs/decisions/0011-producing-to-event-bus-via-settings.rst
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ Status
Context
*******

The initial implementation of the event bus allowed only a single event type to be published to a topic, with details like topic name, consumer group name, and consumer name were configured via code. We weren't sure what the api would finally look like, and whether the event bus config would live as part of the definition of the event, so we just went with explicit code that we could iterate upon.
The initial implementation of the event bus allowed only a single event type to be published to a topic, with details like topic name, consumer group name, and consumer name configured via code. We weren't sure what the api would finally look like, and whether the event bus config would live as part of the definition of the event, so we just went with explicit code that we could iterate upon.

The current implementation of openedx-events does not actually push any events to the underlying implementations like `edx-event-bus-kafka`_ and `edx-event-bus-redis`_. The event-producing application is expected to create a signal handler (since openedx-events subclasses Django signals) to catch the event and push it into the event bus. Some examples of the handlers: `handlers example`_.

Expand Down Expand Up @@ -88,5 +88,5 @@ Consequences
Rejected Alternatives
*********************

* Implementing configurable handlers in the host applications will require repeating the code in each host application.
* Following the current way of using fixed handlers will restricts users ability to combine events in a single topic based on their preference.
* Implementing configurable handlers in the host applications, rather than in the shared library, will require repeating the code in each host application.
* Following the current way of using fixed handlers will restricts users ability to combine events in a single topic based on their preference, as well as stopping users from producing additional defined events to the event bus without code changes.

0 comments on commit fd624f9

Please sign in to comment.