Skip to content

Commit

Permalink
fixup!: merge configs
Browse files Browse the repository at this point in the history
  • Loading branch information
Rebecca Graber committed Oct 4, 2023
1 parent 90f8440 commit 4340f98
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/decisions/0013-new-event-bus-producer-config.rst
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,10 @@ The new EVENT_BUS_PRODUCER_CONFIG will have the following configuration format:
},
}
A new ``merge_configs`` method will be added to openedx-events.event_bus to make it easier to correctly determine the config map from multiple sources.
A new ``merge_publisher_configs`` method will be added to openedx-events.event_bus to make it easier to correctly determine the config map from multiple sources.

Consequences
************

* As long as the implementing IDA calls ``merge_configs``, maintainers can add existing topics to new event_types without having to recreate the whole dictionary
* As long as the implementing IDA calls ``merge_publisher_configs``, maintainers can add existing topics to new event_types without having to recreate the whole dictionary
* There is no ambiguity about whether an event/topic pair is enabled or disabled

0 comments on commit 4340f98

Please sign in to comment.