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

[14.0][FIX] account_financial_report: Don't include unaffected account if accounts are filtered #780

Merged

Conversation

joao-p-marques
Copy link
Member

When generating a General Ledger or a Trial Balance, if some accounts are already filtered in the wizard don't show the information related to the unaffected earnings account

FWP of #779

@Tecnativa
TT29658

ping @pedrobaeza @victoralmau

@pedrobaeza pedrobaeza added this to the 14.0 milestone May 12, 2021
@joao-p-marques joao-p-marques force-pushed the 14.0-fix-account_financial_report-unaffected branch from cadf62f to 8f70217 Compare May 12, 2021 09:02
…ccounts are filtered

When generated a General Ledger or a Trial Balance, if some accounts are already filtered in the wizard don't show the information related to the unaffected earnings account

TT29658
@joao-p-marques joao-p-marques force-pushed the 14.0-fix-account_financial_report-unaffected branch from 8f70217 to 770f034 Compare May 12, 2021 10:26
@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
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-780-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 95b38ab into OCA:14.0 May 17, 2021
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 6f5cd77. 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.

None yet

4 participants