[19.0][MIG]: endpoint#127
Conversation
By putting the model in the routing handler options we allow to: * avoid override of _default_endpoint_options_handler if you only want to search another model * make explicit in the route configuration the model that will be searched
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: web-api-14.0/web-api-14.0-endpoint Translate-URL: https://translation.odoo-community.org/projects/web-api-14-0/web-api-14-0-endpoint/
Alexgars73
force-pushed
the
19.0-mig-endpoint
branch
from
61e3067 to
704d0e3
Compare
| rec._validate_exec_mode() | ||
|
|
||
| def _validate_exec_mode(self): | ||
| validator = getattr(self, "_validate_exec__" + self.exec_mode, lambda x: True) |
There was a problem hiding this comment.
| validator = getattr(self, "_validate_exec__" + self.exec_mode, lambda x: True) | |
| validator = getattr(self, "_validate_exec__" + self.exec_mode, lambda: True) |
the validator is called without arguments, so the default lambda value shuoldn't take any
There was a problem hiding this comment.
this is a pre-existing bug. can you solve in a separate commit so we can easily backport?
|
Hello @Alexgars73 , will you be able to update and fix this PR soon or I can replace it with another one and speed up the migration process? |
|
@yankinmax i added the change that @ivantodorovich request, the dependency still not merged so we cannot do nothing at moment. |
Thanks a lot. I'll try to speed up |
|
Hey @Alexgars73 the |
Alexgars73
force-pushed
the
19.0-mig-endpoint
branch
from
d50f0c2 to
78e63ed
Compare
|
@yankinmax done! |
|
Hello @simahawk can you pls trigger a merge here? |
|
This PR has the |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at a8bd9c5. Thanks a lot for contributing to OCA. ❤️ |
16 participants
Standard migration
Depends on