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

fix(cost center): uniquely show auxiliary cost centers in columns #5982

Conversation

jniles
Copy link
Contributor

@jniles jniles commented Oct 6, 2021

This commit fixes the presentation of the cost center report to only show the allocation ratios for the auxiliary cost centers in the columns.

Closes #5979.

image

This commit fixes the presentation of the cost center report to only
show the allocation ratios for the auxiliary cost centers in the
columns.

Closes IMA-WorldHealth#5979.
@jniles jniles requested a review from mbayopanda October 6, 2021 10:35
@jniles jniles changed the title fix(cost center): uniquely auxiliary in columns fix(cost center): uniquely show auxiliary cost centers in columns Oct 6, 2021
Copy link
Contributor

@mbayopanda mbayopanda left a comment

Choose a reason for hiding this comment

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

Only auxiliary cost centers columns are shown 👍🏽

LGTM

bors r+

@@ -29,8 +29,10 @@
<th>{{translate 'FORM.LABELS.SERVICE'}}</th>
<th>{{translate 'COST_CENTER.DIRECT_COST'}}</th>
{{#each data as | service |}}
{{#unless service.is_principal}}
Copy link
Contributor

Choose a reason for hiding this comment

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

👍🏽

@bors
Copy link
Contributor

bors bot commented Oct 6, 2021

Build succeeded:

@bors bors bot merged commit 4d0f032 into IMA-WorldHealth:master Oct 6, 2021
@jniles jniles deleted the 5979-refactor-hide-principal-cost-centers-in-cost-allocation-report branch October 6, 2021 11:17
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.

Do not show principal cost centers in the columns of step-down cost allocation report
2 participants