Skip to content

Conversation

mostlyobvious
Copy link
Member

@mostlyobvious mostlyobvious commented Nov 29, 2022

Related to the discussion. This is however API-breaking change, even if part of this was unspecified behaviour. We'll delay this change a bit to next major release.

Reverts #1465

Changelog entry:

* Fix:  More explicit and intentional behaviour has been implemented for `Client#publish`,  `Client#append` and `Client#link` methods receiving `nil` as an argument. Publishing, appending or linking `nil` events will result in `ArgumentError` raised. [#1269, #1354, #1453, #1466]

@mostlyobvious mostlyobvious added this to the v3.0.0 milestone Nov 29, 2022
mostlyobvious and others added 23 commits January 30, 2023 11:47
Since we have introduced AggregateRoot::Repository it is
easy to pass event store as a dependency there. No need
to use AggregateRoot::Configuration and default_event_store.

Make it explicte which event store is used. Less "magic".
* no sane way to support event type scheme for extract last part
* no sane way to normalize every event type to method name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants