Added ability to group invoices by PIs #10
Conversation
| remove_billables(merged_dataframe, pi, projects, "non_billable.csv") | ||
| export_pi_billables(billable_projects, args.output_folder) |
There was a problem hiding this comment.
can we also make this optional and triggered by a flag? I think the default behavior should still be to produce the 2 reports i.e. billable and non-billable, but the user can say --export-by-pi and then it should split it out into multiple files. @joachimweyl thoughts?
There was a problem hiding this comment.
that makes sense, as long as it is a flag we know to use and use every month until we decide to be done with it.
There was a problem hiding this comment.
@QuanMPhm pointed out that the billable csv is still written, so I am okay with the current behavior then and don't care much about the flag.
There was a problem hiding this comment.
@naved001 @joachimweyl It should only take minutes for me to make it optional for the script to export the invoice csv by pi or as combined or as both. I'll do what you prefer.
Closes #7, I have modified
process_report.pyto group the merged invoices into ones grouped by PIs. These smaller CSVs will be placed in a folder of your choice, or defaults to a one placed in the working directory