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

[ADD] account_analytic_report: New module account_analytic_report #1249

Closed

Conversation

BernatObrador
Copy link

@BernatObrador BernatObrador commented Nov 7, 2024

New module account_analytic_report

Using this module is straightforward. Follow these steps:

  • Navigate to the Report:
    Go to Invoicing -> Reporting -> Analytic Trial Balance.

  • Customize the Report with Filters:
    Adjust the report using the available options:

    • Group by Analytic Account:
      Groups the results by analytic accounts instead of financial accounts.

    • Show Hierarchy and Limit Hierarchy Level:
      Displays the amounts split by the hierarchy levels of financial accounts.

    • Filter Accounts:
      When used independently (without grouping by analytic accounts or showing hierarchy), the results will be split by both financial accounts.
      Example: Filtering by accounts Test 1 and Test 2:

              | Initial Balance | Test 1   | Test 2   | Ending Balance
      400000  |        0        | $3600    | $2400    |     $6000
      
    • Show Months (Excel export only):
      Enabled when filtering accounts without grouping by analytic accounts or showing hierarchy. It generates a separate sheet in the Excel file for each filtered account, detailing the amounts by month within the selected date range.

cc https://github.com/APSL 162352

@miquelalzanillas @lbarry-apsl @mpascuall @peluko00 @javierobcn @ppyczko please review

@BernatObrador BernatObrador force-pushed the 17.0-add-account_analytic_report branch 3 times, most recently from 4ed0b0a to d434e2d Compare November 8, 2024 14:11
@BernatObrador BernatObrador force-pushed the 17.0-add-account_analytic_report branch 4 times, most recently from 925a0ea to 6e38739 Compare December 11, 2024 13:35
@BernatObrador BernatObrador changed the title [WIP] account_analytic_report: New module account_analytic_report [ADD] account_analytic_report: New module account_analytic_report Dec 11, 2024
@BernatObrador BernatObrador marked this pull request as ready for review December 11, 2024 13:48
Comment on lines 21 to 26
<field name="date_from" required="1" />
<field name="date_to" required="1" />
<field name="fy_start_date" invisible="1" />
</group>
<group name="other_filters">
<field name="plan_id" required="1" />

Choose a reason for hiding this comment

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

Put required on a declaration of the field

@BernatObrador BernatObrador force-pushed the 17.0-add-account_analytic_report branch from 6e38739 to e5b0c72 Compare December 17, 2024 13:26
@BernatObrador
Copy link
Author

Thanks for your comment @peluko00! Could you review again please?

Copy link

@peluko00 peluko00 left a comment

Choose a reason for hiding this comment

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

Works as expected in runboat and code review

@BernatObrador BernatObrador force-pushed the 17.0-add-account_analytic_report branch from e5b0c72 to 16e6378 Compare December 18, 2024 11:52
Copy link

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

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

LGTM, nice work!

@pedrobaeza
Copy link
Member

Shouldn't this be in OCA/account-analytic instead?

@BernatObrador
Copy link
Author

BernatObrador commented Dec 19, 2024

@pedrobaeza You are totally right. Here is the new pr: #730

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants