Skip to content

fix: pass report boolean flags to correct parameters - #20

Draft
ryanduguid wants to merge 1 commit into
XeroAPI:mainfrom
ryanduguid:fix/report-boolean-flags
Draft

fix: pass report boolean flags to correct parameters#20
ryanduguid wants to merge 1 commit into
XeroAPI:mainfrom
ryanduguid:fix/report-boolean-flags

Conversation

@ryanduguid

Copy link
Copy Markdown

Summary

  • pass --standard-layout and --payments-only to their matching Balance Sheet and Profit & Loss SDK parameters
  • correct the Profit & Loss tracking-argument comments to the SDK's order
  • add command-level regression tests for both report commands

The two boolean flags were reversed, so --payments-only requested standard layout and never enabled the intended cash-basis filter.

Testing

  • npm test (132 passed)
  • npm run build

@ryanduguid
ryanduguid marked this pull request as draft August 12, 2026 17:42
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.

1 participant