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

[t141292] avoid m2m multi-company issue #1425

Merged
merged 1 commit into from
Jan 10, 2024

Conversation

BT-etejeda
Copy link

@BT-etejeda BT-etejeda commented Jan 10, 2024

Invalidate cache after calling read method with sudo and before calling write method without sudo

Links to Odoo:
Affected Modules:
ModuleExt
auditlog.py

Invalidate caché after calling read method with sudo and before calling write method without sudo
Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ac5ea0c) 78.84% compared to head (1a98607) 78.85%.
Report is 2 commits behind head on 15.0.project_GE_578.

Additional details and impacted files
@@                 Coverage Diff                  @@
##           15.0.project_GE_578    #1425   +/-   ##
====================================================
  Coverage                78.84%   78.85%           
====================================================
  Files                      313      313           
  Lines                     9815     9816    +1     
  Branches                  1392     1392           
====================================================
+ Hits                      7739     7740    +1     
  Misses                    1788     1788           
  Partials                   288      288           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@BT-mgomez BT-mgomez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Only point is that the coverage have lighly decreased due to this, not sure they will approve or give problems on the OCA when you upload this, let's see.. :)

@BT-etejeda
Copy link
Author

I already made the PR yesterday, and nobody has react :/
This is the URL: OCA#2799

@BT-etejeda BT-etejeda merged commit e10f803 into 15.0.project_GE_578 Jan 10, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants