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.

Sign up for GitHub

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

Jump to bottom

Reports using Enterprise currency #924

Closed
5 of 6 tasks
sfount opened this issue Nov 23, 2016 · 1 comment · Fixed by #935 or #936
Closed
5 of 6 tasks

Reports using Enterprise currency #924

sfount opened this issue Nov 23, 2016 · 1 comment · Fixed by #935 or #936
Labels
Report

Comments

@sfount
Copy link
Contributor

sfount commented Nov 23, 2016 •
edited by jniles
Loading

This issue is to track ensuring all reports are provided in the enterprises set currency.

Required refactors

  • Patient Invoice
  • Cashflow (Requires controller refactor/ simplification for future maintenance)
  • Patient financial activity
  • Customer Debts
  • Income vs. Expense
  • Accounts Report
@sfount sfount mentioned this issue Nov 23, 2016
Merged
@jniles jniles added this to the Finance Implementation milestone Nov 24, 2016
@jniles jniles added the Report label Nov 24, 2016
@jniles
Copy link
Collaborator

jniles commented Nov 24, 2016

I'll look into the accounts report.

@jniles jniles mentioned this issue Nov 24, 2016
Merged
4 tasks
jniles pushed a commit to jniles/bhima that referenced this issue Nov 24, 2016
fix(report): improve account report
974bfd6 
This commit improves the account report by hooking up a loading
indicator to the form, using a ui-select for accounts, and ensuring the
report renders in the enterprise currency.

Partially addresses Third-Culture-Software#924.
jniles pushed a commit to jniles/bhima that referenced this issue Nov 25, 2016
fix(report): account report uses ent currency
7e618b8 
This commit fixes account report to use the enterprise currency in
rendering totals.  It also migrates the account selection to a ui-select
for good measure.  Finally, the loading indicator on the form is now
  hooked up.

Partially fixes Third-Culture-Software#924.
This was referenced Nov 25, 2016
Merged
Merged
jniles pushed a commit to jniles/bhima that referenced this issue Nov 26, 2016
fix(report): account report uses ent currency
ebdb798 
This commit fixes account report to use the enterprise currency in
rendering totals.  It also migrates the account selection to a ui-select
for good measure.  Finally, the loading indicator on the form is now
  hooked up.

Partially fixes Third-Culture-Software#924.
jniles pushed a commit to jniles/bhima that referenced this issue Nov 27, 2016
fix(report): account report uses ent currency
b6af238 
This commit fixes account report to use the enterprise currency in
rendering totals.  It also migrates the account selection to a ui-select
for good measure.  Finally, the loading indicator on the form is now
  hooked up.

Partially fixes Third-Culture-Software#924.
sfount pushed a commit that referenced this issue Nov 27, 2016
@sfount
fix(report): account report uses ent currency
aaada58 
This commit fixes account report to use the enterprise currency in
rendering totals.  It also migrates the account selection to a ui-select
for good measure.  Finally, the loading indicator on the form is now
  hooked up.

Partially fixes #924.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Report
Projects
None yet
Milestone
No milestone
2 participants
@jniles @sfount