Refactored the HU-BU invoice

CCI-MOC / process_csv_report

Some scripts to help process our billing reports

0 stars 3 forks source link

Refactored the HU-BU invoice #70

Closed QuanMPhm closed 3 months ago

QuanMPhm commented 4 months ago

Closes #68. Not necessarily dependent on #61, but is part of the overall refactor process. Aside from the addition of hu_bu_invoice.py, I decided to cache the results of get_invoice_bucket() using functools

While refactoring, I noticed that I didn't add code to upload the BU Internal invoice to S3, and also forgot to remove some functions from process_report.py that have been moved to utils.py. @naved001 @knikolla Should these be part of a separate commit or issue?

joachimweyl commented 4 months ago

@QuanMPhm what are the next steps to getting this merged?

QuanMPhm commented 4 months ago

@joachimweyl This PR is dependent on #61, so I'll have to wait for that to resolve first.