Skip to content

Conversation

@mybracho
Copy link
Collaborator

… 1.1

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds two new report subsystems (Fixed Assets and Accounting) to the Reports subsystem, along with their associated command handlers for opening various reports.

  • Adds "Fixed Assets" subsystem with 4 reports (transfer cost, inventory book, depreciation statement, book value)
  • Adds "Accounting" subsystem with 5 reports (trial balance, trial balance by account, account card, account analysis, P&L report)
  • Registers all 9 common commands in the main configuration

Reviewed changes

Copilot reviewed 24 out of 24 changed files in this pull request and generated 13 comments.

Show a summary per file
File Description
IRP/src/Subsystems/Reports/Reports.mdo Registers FixedAssets and Accounting as new subsystems under Reports
IRP/src/Subsystems/Reports/Subsystems/FixedAssets/FixedAssets.mdo Defines FixedAssets subsystem with 4 report commands
IRP/src/Subsystems/Reports/Subsystems/FixedAssets/CommandInterface.cmi Command interface definition for FixedAssets subsystem
IRP/src/Subsystems/Reports/Subsystems/Accounting/Accounting.mdo Defines Accounting subsystem with 5 report commands
IRP/src/Subsystems/Reports/Subsystems/Accounting/CommandInterface.cmi Command interface definition for Accounting subsystem
IRP/src/Configuration/Configuration.mdo Registers all 9 new common commands in the main configuration
IRP/src/CommonCommands/OpernReport_A0013_AccountAnalysis/* Command definition and handler for Account Analysis report (contains naming typo)
IRP/src/CommonCommands/OpenReport_R8510B_BookValueOfFixedAsset/* Command definition and handler for Book Value report (contains incorrect report reference)
IRP/src/CommonCommands/OpenReport_F0012_DepreciationStatement/* Command definition and handler for Depreciation Statement report
IRP/src/CommonCommands/OpenReport_F0011_InventoryBook/* Command definition and handler for Inventory Book report
IRP/src/CommonCommands/OpenReport_F0010_FixedAssetsTransferCost/* Command definition and handler for Fixed Assets Transfer Cost report
IRP/src/CommonCommands/OpenReport_A0014_AccountingPLReport/* Command definition and handler for Accounting P&L Report
IRP/src/CommonCommands/OpenReport_A0012_AccountCard/* Command definition and handler for Account Card report
IRP/src/CommonCommands/OpenReport_A0011_TrialBalanceByAccount/* Command definition and handler for Trial Balance by Account report
IRP/src/CommonCommands/OpenReport_A0010_TrialBalance/* Command definition and handler for Trial Balance report

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

mybracho and others added 14 commits December 28, 2025 12:53
…odule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…t/CommandModule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…mandModule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…/CommandModule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…CommandModule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ndModule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…/CommandModule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…dule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…CommandModule.bsl

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…Report_A0013_AccountAnalysis.mdo

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

#IRP-792 Add Fixed Assets and Accounting Reports to Reports Subsystem 1.1

2 participants