-
-
Notifications
You must be signed in to change notification settings - Fork 297
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
[16.0][MIG] - rest_log #354
Conversation
64e6425
to
86724c7
Compare
@sbejaoui Can you prepare a PR base on this one with the improvements listed below: These come from https://github.com/acsone/rest-framework/commits/10.0-backport-from-14.0/rest_log |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (Code review)
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
@lmignon your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-354-by-lmignon-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
1 similar comment
@lmignon your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-354-by-lmignon-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
We need to finish #360 to get the 16.0 branch green again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see some fixes: can you please check v14?
AFAIR we made some fix/imp there (the Response obj for instance).
64ec097
to
d7998a4
Compare
Provide a hook to customize stored params. At the same time, assume passwords must be always cleaned up.
Some endpoints will not return a dict as expected by the ``_dispatch_with_db_logging`` method (and the other methods called within), but an object of type ``odoo.http.Response``. When that happens, the instance fails to handle it while creating logs (ie: tries to call ``json.dumps()`` on such object, which is not allowed). This patch should fix the issue.
This way we can hook an manipulate data w/out having to decode from JSON back and forth.
36ee037
to
daab30b
Compare
/ocabot merge patch |
This PR looks fantastic, let's merge it! |
It looks like something changed on |
Congratulations, your PR was merged at 883c8fb. Thanks a lot for contributing to OCA. ❤️ |
No description provided.