[18.0][IMP] edi_core: add global events handling via edi.conf#275
Merged
Conversation
OCA-git-bot
added
series:18.0
mod:edi_core_oca
Contributor
|
Hi @etobella, |
simahawk
force-pushed
the
18-core-listeners
branch
2 times, most recently
from
1b35b67 to
b8aa554
Compare
Global configurations can be used to catch generic events not bound to specific partner releations. As such, they are the core framework replacement for component events. Co-authored-by: Arnau <arnau.cruz@forgeflow.com>
Co-authored-by: Arnau <arnau.cruz@forgeflow.com>
* full matching by type, backend and model * full test coverage
No comparison if backend is not set or type is not set.
simahawk
force-pushed
the
18-core-listeners
branch
from
b8aa554 to
40e51cf
Compare
Contributor
Author
|
@ArnauCForgeFlow @etobella could you please have a look at this? 😇 |
yankinmax
reviewed
May 18, 2026
yankinmax
approved these changes
May 18, 2026
The event name would be prefixed 2 times when calling super.
2 tasks
ArnauCForgeFlow
approved these changes
May 19, 2026
Contributor
ArnauCForgeFlow
left a comment
ArnauCForgeFlow
left a comment
There was a problem hiding this comment.
LGTM. Reimplemented edi_storage_oca done/error listener on top of this, works nicely
Contributor
|
This PR has the |
Contributor
Author
|
/ocabot merge minor |
Contributor
|
Hey, thanks for contributing! Proceeding to merge this for you. |
Contributor
|
Congratulations, your PR was merged at 9e2a827. Thanks a lot for contributing to OCA. ❤️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Replaces #232
See atomic commits and updated docs for details 😉
Long story short: we now have global events, that means we can avoid using component_event if not required.