Skip to content
This repository was archived by the owner on Nov 30, 2025. It is now read-only.

319 replacing redundant waterloo api logic in relation to converttermcodetotermname#327

Merged
sai3000pro merged 3 commits intomainfrom
319-replacing-redundant-waterloo-api-logic-in-relation-to-converttermcodetotermname
Feb 13, 2025
Merged

319 replacing redundant waterloo api logic in relation to converttermcodetotermname#327
sai3000pro merged 3 commits intomainfrom
319-replacing-redundant-waterloo-api-logic-in-relation-to-converttermcodetotermname

Conversation

@sai3000pro
Copy link
Contributor

Commented out the functions that no longer worked due to the API not working. Appears to work normally on my local version! We might want to add in validation that works later, though.

@sai3000pro
Copy link
Contributor Author

I just commented the code out for now - I can delete it if that's more preferred!

@some-molasses
Copy link
Member

yeah let's delete - keeps things cleaner that way! if we need it again we can just pull from the git history

Copy link
Member

@some-molasses some-molasses left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approving anyway but deleting is preferred!

@sai3000pro sai3000pro merged commit 91b9eb8 into main Feb 13, 2025
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replacing redundant Waterloo API logic in relation to getting term names from term codes

2 participants