-
Notifications
You must be signed in to change notification settings - Fork 17
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
[ADD] add new module website_rebrand_coopiteasy #313
Conversation
625f482
to
841d176
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## 12.0 #313 +/- ##
=======================================
Coverage 69.98% 69.98%
=======================================
Files 153 153
Lines 2012 2012
Branches 366 366
=======================================
Hits 1408 1408
- Misses 565 566 +1
+ Partials 39 38 -1 ☔ View full report in Codecov by Sentry. |
1d27794
to
318a367
Compare
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.
More a comment than a review : "Odoo by Coop IT Easy" sounds like Odoo is a product from Coop IT Easy. I would be more precise, for instance "Odoo, supplied by Coop IT Easy".
indeed. i had the same impression, and your suggestion sounds very good and accurate. other options i think of would be “provided by” or “managed by”. |
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
a0ce9ae
to
c7a17fc
Compare
c7a17fc
to
df12d7a
Compare
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at dd07f0f. Thanks a lot for contributing to coopiteasy. ❤️ |
description
add a new module to replace the default odoo website branding by a coop it easy branding.
odoo task
t12258