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

[MIG] Migrate module disable_odoo_online to v14. #35

Merged
merged 18 commits into from
Jan 25, 2021

Conversation

dsolanki-initos
Copy link
Contributor

No description provided.

tungocbui and others added 17 commits December 18, 2020 10:45
…. 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/
[FIX] pre-commit

[FIX] Remove use of api.multi

[FIX] pre-commit

[FIX] pre-commit

[FIX] pre-commit

[FIX] pre-commit
@dsolanki-initos dsolanki-initos force-pushed the 14.0-mig-disable_odoo_online branch 2 times, most recently from a32d37c to b521722 Compare December 18, 2020 05:23
@dsolanki-initos
Copy link
Contributor Author

Hello @ALL
Can anyone please review the PR?

disable_odoo_online/models/publisher_warranty_contract.py Outdated Show resolved Hide resolved
disable_odoo_online/views/ir_ui_menu.xml Outdated Show resolved Hide resolved
@dsolanki-initos
Copy link
Contributor Author

@astirpe I have resolved the changes suggested.

Copy link

@bosd bosd left a comment

Choose a reason for hiding this comment

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

Functional review, LGTM 👍

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza pedrobaeza added this to the 14.0 milestone Jan 25, 2021
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-35-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c6269f4 into OCA:14.0 Jan 25, 2021
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 2cef039. Thanks a lot for contributing to OCA. ❤️

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.