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

11.0 fix report #31150

Closed
wants to merge 1 commit into from
Closed

11.0 fix report #31150

wants to merge 1 commit into from

Conversation

etobella
Copy link
Contributor

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

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

@etobella etobella changed the base branch from 12.0 to 11.0 February 15, 2019 13:01
@robodoo robodoo added seen 🙂 CI 🤖 Robodoo has seen passing statuses labels Feb 15, 2019
@JordiBForgeFlow
Copy link
Contributor

@qdp-odoo what do you think about this?

@qdp-odoo
Copy link
Contributor

qdp-odoo commented Mar 6, 2019

@robodoo r+

@robodoo robodoo added the r+ 👌 label Mar 6, 2019
@robodoo
Copy link
Contributor

robodoo commented Mar 6, 2019

Because this PR has multiple commits, I need to know how to merge it:

  • merge to merge directly, using the PR as merge commit message
  • rebase-merge to rebase and merge, using the PR as merge commit message
  • rebase-ff to rebase and fast-forward

@MiquelRForgeFlow
Copy link
Contributor

@qdp-odoo

robodoo pushed a commit that referenced this pull request Mar 13, 2019
closes #31150

Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>
@xmo-odoo
Copy link
Collaborator

@mreficent fixed the flag in the backend, mergebot should stop believing this PR has multiple commits (also fixed the bit where retargeting the PR between branches didn't update that flag, cf runbot/odoo#82 so this should not happen again)

@robodoo
Copy link
Contributor

robodoo commented Mar 13, 2019

Merged, thanks!

@robodoo robodoo closed this Mar 13, 2019
Tonow-c2c added a commit to Tonow-c2c/OCB that referenced this pull request Jan 28, 2020
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
Tonow-c2c added a commit to camptocamp/odoo that referenced this pull request Jan 28, 2020
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
Tonow-c2c added a commit to camptocamp/odoo that referenced this pull request Jan 30, 2020
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
Tonow-c2c added a commit to camptocamp/odoo that referenced this pull request Jan 30, 2020
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
AaronHForgeFlow pushed a commit to ForgeFlow/odoo that referenced this pull request Jun 3, 2020
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
santostelmo pushed a commit to camptocamp/odoo that referenced this pull request Aug 17, 2020
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
gkliska pushed a commit to decodio/odoo10 that referenced this pull request Nov 2, 2021
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/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
yankinmax pushed a commit to camptocamp/odoo that referenced this pull request Jan 17, 2022
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
i-vyshnevska pushed a commit to camptocamp/odoo that referenced this pull request Jan 27, 2022
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
i-vyshnevska pushed a commit to camptocamp/odoo that referenced this pull request Jan 27, 2022
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
i-vyshnevska pushed a commit to camptocamp/odoo that referenced this pull request Feb 10, 2022
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
i-vyshnevska pushed a commit to camptocamp/odoo that referenced this pull request May 25, 2022
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants