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

[13.0][BKP][ADD]edi_endpoint_oca: rename it to edi_endpoint #1062

Open
wants to merge 41 commits into
base: 13.0
Choose a base branch
from

Conversation

GuillemCForgeFlow
Copy link

Based on changes added in #1059. Now creating the PR only backporting the edi_endpoint_oca module, renamed here in v13 to be in line with the edi_webservice from v13: https://github.com/OCA/edi/tree/13.0/edi_webservice

cc @ForgeFlow

simahawk and others added 30 commits November 7, 2024 15:50
Block archiving of backends if they have active endpoints exposed.
* more efficient w/ read_group
* consider only active records explicitly
I don't know why but  when test_edi_endpoint runs before these tests
the rollback of the exception catched within the test `test_archive_check`
make the controller lookup fail.
Since the whole routing registry machinery is going to be refactored
in OCA#633
let's survive w/ this forced registration for now.
Former version of `endpoint_route_handler` had a major flaw:

routing rule registry was not properly shared across workers
forcing us to restart the instance to make sure all envs were inline w/
it.

This change adapts edi_endpoint_oca to the new version which contains
some refactoring.
It's very handy to know which endpoint generated a record.
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.

7 participants