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

[15.0][IMP] website_odoo_debranding: Avoid deprecation warning with 'Enviro… #901

Merged
merged 1 commit into from
Jun 6, 2022

Conversation

dhx
Copy link

@dhx dhx commented Jun 6, 2022

…nment.manage()'

Closes #899

(now without the update of the version)

@pedrobaeza
Copy link
Member

You can forced push in the same branch for not needing to open a new PR, but this is only nitpicking for saving some resources.

Let's fast-track it as straight-forward:

/ocabot merge patch

No need also to open the issue and the PR if you plan to directly fix it. Just do the PR exposing the issue.

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-901-by-pedrobaeza-bump-patch, awaiting test results.

@pedrobaeza pedrobaeza changed the title [IMP] website_odoo_debranding: Avoid deprecation warning with 'Enviro… [15.0][IMP] website_odoo_debranding: Avoid deprecation warning with 'Enviro… Jun 6, 2022
@OCA-git-bot OCA-git-bot merged commit c0d1926 into OCA:15.0 Jun 6, 2022
@OCA-git-bot
Copy link
Contributor

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

@dhx dhx deleted the 15.0-env-fix-website-odoo-debranding-2 branch June 8, 2022 09:27
@dhx
Copy link
Author

dhx commented Jun 8, 2022

You can forced push in the same branch for not needing to open a new PR, but this is only nitpicking for saving some resources.

No need also to open the issue and the PR if you plan to directly fix it. Just do the PR exposing the issue.

Thanks for clarifying!

@rafaelbn
Copy link
Member

/ocabot migration website_odoo_debranding

@OCA-git-bot OCA-git-bot mentioned this pull request Sep 27, 2022
12 tasks
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.

4 participants