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.

Sign up for GitHub

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

Jump to bottom

[15.0][IMP] server-ux : test performance improvement #739

Merged
merged 3 commits into from
Oct 25, 2023
Merged

[15.0][IMP] server-ux : test performance improvement #739

merged 3 commits into from
Oct 25, 2023

Conversation

josep-tecnativa
Copy link

CC @Tecnativa TT45458

@OCA-git-bot
Copy link
Contributor

Hi @LoisRForgeFlow,
some modules you are maintaining are being modified, check this out!

@josep-tecnativa
Copy link
Author

ping @pedrobaeza

@pedrobaeza pedrobaeza added this to the 15.0 milestone Oct 25, 2023
pedrobaeza
pedrobaeza approved these changes Oct 25, 2023
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit ec32eae into OCA:15.0 Oct 25, 2023
5 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 15.0-imp-tests branch October 25, 2023 11:42
@bosd
Copy link
Contributor

bosd commented Nov 4, 2023

I'm a bit confused by the comment:

        # Remove this variable in v16 and put instead:
        # from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT

As the modules already have been migrated to V16 😀

@bosd bosd mentioned this pull request Nov 4, 2023
Merged
@pedrobaeza
Copy link
Member

It was a mistake as these improvements were done in parallel in v15 and v16 and cherry-picking commits. You can do another PR doing such substitution.

This was referenced Nov 29, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chienandalu chienandalu chienandalu approved these changes

@pedrobaeza pedrobaeza pedrobaeza approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.
5 participants
@josep-tecnativa @OCA-git-bot @bosd @pedrobaeza @chienandalu