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

V16 partner_hide_technical_*** #185

Conversation

legalsylvain
Copy link
Member

[ADD] partner_hide_technical_company_calendar: glue module to make the module partner_hide_technical_company working when calendar is installed

…e module partner_hide_technical_company working when calendar is installed
@legalsylvain legalsylvain added this to the 16.0 milestone Feb 14, 2025
Copy link

codecov bot commented Feb 14, 2025

Codecov Report

Attention: Patch coverage is 96.91358% with 5 lines in your changes missing coverage. Please review.

Project coverage is 90.23%. Comparing base (675aaa1) to head (2fc3a4a).
Report is 52 commits behind head on 16.0.

Files with missing lines Patch % Lines
..._hide_technical_fix_calendar/models/res_partner.py 76.92% 0 Missing and 3 partials ⚠️
...tner_hide_technical_abstract/models/res_partner.py 94.11% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             16.0     #185      +/-   ##
==========================================
+ Coverage   89.34%   90.23%   +0.88%     
==========================================
  Files          93      106      +13     
  Lines        1145     1259     +114     
  Branches      108      115       +7     
==========================================
+ Hits         1023     1136     +113     
+ Misses         91       89       -2     
- Partials       31       34       +3     

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

@legalsylvain legalsylvain force-pushed the 16.0-REF-ADD-partner_hide_technical_employee branch 3 times, most recently from 4af45d7 to 9f612c1 Compare February 14, 2025 15:49
…in all modules

[REF] Add partner_hide_technical_fix_calendar and partner_hide_technical_fix_calendar_test.
So removing, partner_hide_technical_user_calendar, partner_hide_technical_employee_calendar
and partner_hide_technical_company_calendar
@legalsylvain legalsylvain force-pushed the 16.0-REF-ADD-partner_hide_technical_employee branch from 9f612c1 to 2fc3a4a Compare February 14, 2025 16:58
@legalsylvain
Copy link
Member Author

/ocabot merge major

@github-grap-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-185-by-legalsylvain-bump-major, awaiting test results.

@github-grap-bot github-grap-bot merged commit ee2e0b5 into grap:16.0 Feb 17, 2025
4 checks passed
@github-grap-bot
Copy link
Contributor

Congratulations, your PR was merged at 6c166fc. Thanks a lot for contributing to grap. ❤️

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.

2 participants