Skip to content
This repository has been archived by the owner on Apr 22, 2024. It is now read-only.

Refactor CSV/DB writing with logging in course_inventory #149

Open
ssciolla opened this issue May 12, 2020 · 0 comments
Open

Refactor CSV/DB writing with logging in course_inventory #149

ssciolla opened this issue May 12, 2020 · 0 comments
Labels
Canvas 📈 enhancement New feature or request

Comments

@ssciolla
Copy link
Contributor

ssciolla commented May 12, 2020

The code that writes data to a CSV or the DB in course_inventory/inventory.py and logs before and after is quite repetitive and can likely be made into function(s) or class(es) to reduce duplication and errors.

Note that this is pretty closely tied to whatever method we chose to use for writing to the DB (pandas, SQLAlchemy, etc.), so some consideration should be given to whether the current method is what we will use going forward.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Canvas 📈 enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant