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

Updated Open Debtors report #6243

Merged

Conversation

jmcameron
Copy link
Collaborator

Finances > Report > Open Debtors
(formerly named "Debtors with Unpaid Debts"

Adds multi-currency support.

Closes #6178

TESTING

  • Try the report with with different combinations of options, especially currency, report date limit

@jmcameron jmcameron force-pushed the update-open-debtors-report branch 2 times, most recently from 265a69f to 5ae5bed Compare January 4, 2022 23:17
@jmcameron
Copy link
Collaborator Author

bors try

bors bot added a commit that referenced this pull request Jan 4, 2022
@bors
Copy link
Contributor

bors bot commented Jan 4, 2022

try

Build failed:

@jniles
Copy link
Collaborator

jniles commented Jan 5, 2022

bors try

bors bot added a commit that referenced this pull request Jan 5, 2022
@bors
Copy link
Contributor

bors bot commented Jan 5, 2022

try

Build succeeded:

@jniles
Copy link
Collaborator

jniles commented Jan 5, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Jan 5, 2022

Build succeeded:

@bors bors bot merged commit c6b605c into Third-Culture-Software:master Jan 5, 2022
@jmcameron jmcameron deleted the update-open-debtors-report branch January 5, 2022 15:15
bors bot added a commit that referenced this pull request Jan 8, 2022
6254: Fixed missing code in previous update r=jniles a=jmcameron

Fix an error in PR #6243.
Somehow, code that was in the handlebars file got lost in the version that got released.  This fix adds that code back in.

**TESTING**
- Use production database
- Try the report: Finance > Report > Open Debtors
   - Try it in the enterprise currency (default), and note a few numbers including the final total at the bottom of the report.   Also, no exchange rate should be listed under the title.
   - Try it with some other currency and verify that the new numbers are correct.  The exchange rate used should be listed at the top of the report (under the title).

Co-authored-by: Jonathan Cameron <jmcameron@gmail.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.

Updates for the Debtors with Unpaid Debts report
2 participants