@redconfetti, there are still a number of snake_case variables in here that @christianv will probably have to end up fixing. Sorry to make the comment after the Pr was merged.  search_type, search_text, user_added, canvas_course_id.  Maybe it's not a good idea to do this in the same Pr, especially with variables you haven't changed yourself, but with the ones you do change, like search_type, maybe we can lend a hand.