Open jonespm opened 10 months ago
The account ID 1 is hardcoded here which looks like a bug, but if you pass another term ID it returns an error so this is actually correct.
I'd add a comment in the code describing why this hardcoded 1 value was used.
https://github.com/tl-its-umich-edu/tool-migration/blob/cd7db6dc2f8e810925b854a4dbeb8b87fbc56f66/migration/manager.py#L47
The account ID 1 is hardcoded here which looks like a bug, but if you pass another term ID it returns an error so this is actually correct.
I'd add a comment in the code describing why this hardcoded 1 value was used.
https://github.com/tl-its-umich-edu/tool-migration/blob/cd7db6dc2f8e810925b854a4dbeb8b87fbc56f66/migration/manager.py#L47