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
SAW - Direct Cost Made Configurable #861
Conversation
…osts method for price [#134879983]
…ine dependent on USE_INDIRECT_COST in service calendar [#134879983]
@@ -87,11 +81,6 @@ def display_total_direct_cost_per_study_pppvs service_request | |||
currency_converter sum | |||
end | |||
|
|||
def display_total_indirect_cost_per_study_pppvs service_request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this being used at app/views/service_requests/calendar_totals.html.haml:44
?
@@ -116,11 +105,6 @@ def display_total_direct_cost_per_study_otfs service_request, line_items | |||
currency_converter sum | |||
end | |||
|
|||
def display_total_indirect_cost_per_study_otfs service_request, line_items |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto. app/views/service_requests/calendar_totals.html.haml:62
@@ -139,11 +123,6 @@ def display_grand_total_direct_costs service_request, line_items | |||
currency_converter sum | |||
end | |||
|
|||
def display_grand_total_indirect_costs service_request, line_items |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
app/views/service_requests/calendar_totals.html.haml:81
app/views/service_requests/review/_review_totals.html.haml:32
@@ -154,11 +133,6 @@ def display_study_grand_total_direct_costs protocol, service_request | |||
currency_converter sum | |||
end | |||
|
|||
def display_study_grand_total_indirect_costs protocol, service_request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
app/views/service_requests/show.xlsx.axlsx:277
app/views/service_requests/show.xlsx.axlsx:279
I added the display_indirect_costs helper methods back for now, but I should check that Wenjun wants indirect totals to be displayed anywhere before properly removing them. |
Total Direct Costs are only displayed when USE_INDIRECT_COST is true, Total Indirect Costs are no longer displayed at all.