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

accountingApi.getReportBankSummary can't take Date parameters. #561

Open
DindoLeonard opened this issue Nov 26, 2021 · 1 comment
Open
Assignees

Comments

@DindoLeonard
Copy link

SDK you're using (please complete the following information):
-"xero-node": "^4.17.0"

Describe the bug
accountingApi.getReportBankSummary can't take Date parameters.

To Reproduce
Steps to reproduce the behavior:

  1. use xero.accountingApi.getReportBankSummary() function
  2. Follow this documentation. https://xeroapi.github.io/xero-node/accounting/index.html#api-Accounting-getReportBankSummary
  3. Try adding a date as second parameter with a type Date, it won't take it and will give an error.

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
Capture

@Eteroa123 Eteroa123 self-assigned this Mar 2, 2022
@RettBehrens
Copy link
Contributor

Confirmed the API and SDK expect a string date param, the bug sits within the code generator template and will be fixed upstream so the generated language specific docs are correct on next release. Apologies for the inconvenience!

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

No branches or pull requests

3 participants