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

[FIX][10.0] context on reports #904

Merged
merged 1 commit into from
Feb 11, 2020
Merged

Conversation

Tonow-c2c
Copy link

Description of the issue/feature this PR addresses:

Current behavior before PR:

  • Fixes the reports

Desired behavior after PR is merged:

  • The original context is not deleted, only updated

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Description of the issue/feature this PR addresses: the context on some reports is reseted inside it. It should be addressed kwargs
Idem as odoo#31150

Current behavior before PR: Fixes the reports

Desired behavior after PR is merged: The original context is not deleted, only updated

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
@StefanRijnhart
Copy link
Member

/ocabot merge

@OCA-git-bot
Copy link

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 10.0-ocabot-merge-pr-904-by-StefanRijnhart-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Feb 11, 2020
Signed-off-by StefanRijnhart
@OCA-git-bot
Copy link

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). 🤖

@OCA-git-bot
Copy link

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

@OCA-git-bot OCA-git-bot merged commit ec96a24 into OCA:10.0 Feb 11, 2020
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