Closed helper08 closed 3 years ago
Seems good enough. I assume you did some amount of testing and made sure you didnt break anything obvious. As long as thats true, feel free to mark the one conversation still open as resolved and merge.
I do think you missed some docstrings, but i dont think it matters thaaaat much
These changes will require some changes made to the database. Nothing major, just have to change the titles of each of the positions to their slug versions rather than their Title Case string versions. (ex. Secretary into secretary, Philanthropy Chair into philanthropy-chair, and Vice President of Health and Safety into vphs)