Skip to content

[16.0][IMP] account_financial_report: Add grouping by analytical account in Trial balance#1243

Merged
OCA-git-bot merged 2 commits into
OCA:16.0from
Tecnativa:16.0-imp-account_financial_report-TT48969
Nov 26, 2024
Merged

[16.0][IMP] account_financial_report: Add grouping by analytical account in Trial balance#1243
OCA-git-bot merged 2 commits into
OCA:16.0from
Tecnativa:16.0-imp-account_financial_report-TT48969

Conversation

@victoralmau

Copy link
Copy Markdown
Member

FWP from 15.0: #1151

Changes done:

  • Change t-esc and t-raw to t-out fromTrial Balance report
  • Refactor from Trial Balance
  • Add grouping by analytical account in Trial balance

ejemplo

Please @pedrobaeza can you review it?

@Tecnativa TT48969

@pedrobaeza pedrobaeza added this to the 16.0 milestone Oct 29, 2024

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested on runboat.

As there's now the possibility of having more than one analytic account in the analytic field, we should put a warning banner in the wizard form to specify that there can be duplicated amounts due to that.

@victoralmau victoralmau force-pushed the 16.0-imp-account_financial_report-TT48969 branch from 6828e15 to 97ada3c Compare October 30, 2024 08:31
@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot merge minor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 7b4ad83 into OCA:16.0 Nov 26, 2024
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

4 participants