-
Notifications
You must be signed in to change notification settings - Fork 7
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 fiscal_company_base #54
Conversation
bc994dc
to
86db237
Compare
86db237
to
dfba0a5
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 16.0 #54 +/- ##
=======================================
Coverage ? 92.85%
=======================================
Files ? 9
Lines ? 182
Branches ? 42
=======================================
Hits ? 169
Misses ? 7
Partials ? 6 ☔ View full report in Codecov by Sentry. |
/ocabot migration fiscal_company_base |
47b8b93
to
a388dbc
Compare
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
- full refactor of code. - create a dedicated OCA/multi-company module 'res_company_access_all_childs' to isolate code. - refactor mixin. merge 'fiscal.child.check.mixin' and 'fiscal.mother.check.mixin' - Add test for mixin models
a388dbc
to
f0fa380
Compare
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 34221ff. Thanks a lot for contributing to grap. ❤️ |
No description provided.