-
-
Notifications
You must be signed in to change notification settings - Fork 143
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
[13.0][MIG] disable_odoo_online #15
Conversation
…. Update the version in the file __openerp__.py
…n. (#583) * [PORT] disable_odoo_online. [REF] remove obsolete code. OCA convention. * [FIX] restauring old feature * [IMP] do not disable some feature for Enterprise Edition. [REF] some improvements
…doo community only (OCA#4) * [DOC] disable_odoo_online: clarify docs -- one of the features is applied only in odoo community
* [REF] Changes template used to modificate user preferences menu * [IMP] Includes app menu from settings in technical parameters menu
Currently translated at 100.0% (1 of 1 strings) Translation: server-brand-12.0/server-brand-12.0-disable_odoo_online Translate-URL: https://translation.odoo-community.org/projects/server-brand-12-0/server-brand-12-0-disable_odoo_online/fr/
Currently translated at 100.0% (1 of 1 strings) Translation: server-brand-12.0/server-brand-12.0-disable_odoo_online Translate-URL: https://translation.odoo-community.org/projects/server-brand-12-0/server-brand-12-0-disable_odoo_online/es/
Currently translated at 100.0% (1 of 1 strings) Translation: server-brand-12.0/server-brand-12.0-disable_odoo_online Translate-URL: https://translation.odoo-community.org/projects/server-brand-12-0/server-brand-12-0-disable_odoo_online/pt_BR/
Currently translated at 100.0% (1 of 1 strings) Translation: server-brand-12.0/server-brand-12.0-disable_odoo_online Translate-URL: https://translation.odoo-community.org/projects/server-brand-12-0/server-brand-12-0-disable_odoo_online/hr/
b903549
to
6a5d171
Compare
/ocabot merge |
What a great day to merge this nice PR. Let's do it! |
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 13.0-ocabot-merge-pr-15-by-pedrobaeza-bump-no. 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. |
Can you please rebase and pass pre-commit? |
This PR has the |
Fixes OCA#15 Check that `domain[2]` is `str` because in some cases it's an `int` and `list` operations don't work on `int`s.
Can this be merged? |
Have you rebased? |
Yes |
Atleast the migration commit is rebased. |
Let's see if it works /ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 13.0-ocabot-merge-pr-15-by-pedrobaeza-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. |
Coming back to #15 (comment) then. |
6a5d171
to
fde407a
Compare
@pedrobaeza fixed it, let see if it passes. |
fde407a
to
cbbcd96
Compare
[FIX] pre-commit [FIX] Remove use of api.multi [FIX] pre-commit [FIX] pre-commit [FIX] pre-commit [FIX] pre-commit
cbbcd96
to
3e476b1
Compare
@pedrobaeza Can you retry, please. |
/ocabot merge |
Hi @pedrobaeza. Your command failed:
Ocabot commands
More information
|
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 4bda6dc. Thanks a lot for contributing to OCA. ❤️ |
No description provided.