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
Sj service request export changes #783
Conversation
@@ -136,6 +136,12 @@ def display_total_cost_per_study_otfs service_request, line_items | |||
############################################# | |||
# Grand Totals | |||
############################################# | |||
def display_ssr_grand_total sub_service_request | |||
sum = 0 |
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.
Is this assignment necessary?
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.
I'm not 100% sure, but if you look at the file, it's written the same way all the other methods there are, I was going for consistency (and they all need revisited anyways, as another story).
@@ -174,6 +174,9 @@ wb.add_worksheet(name: "Review") do |sheet| | |||
line_items_visit.line_item.pricing_scheme = 'effective' if @admin_offset | |||
|
|||
service_row << display_visit_based_direct_cost(line_items_visit) | |||
puts '#' * 50 |
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.
puts
No description provided.