DenverCoder1 / jct-discord-bot

Bot for JCT ESP CompSci Discord Server
MIT License
11 stars 1 forks source link

Add a description field to people in the email registry #292

Open abrahammurciano opened 2 years ago

abrahammurciano commented 2 years ago

It would be great to allow an optional description field for things like the person's position like "Dean" or "rector" for people in the administration, to achieve a similar effect as linking teachers to their channels.

This would be useful since it's not immediately obvious what role an administration member has (as it is with teachers), and it would be easier to search administration members by their position rather than their name (esp. if the user doesn't know the name of the person with the position in question)

avipars commented 1 year ago

From discord chat: I looked at the code and it seems like we'd need to add a field into the sql table (and maybe even do a migration) 😭

if we add descriptions which would require some modification of the sql structure anyways, would it be worth adding in more optional fields... I know that knowing who the course coordinator would be helpful (but apparently it changes year to year)... I would think that you just have a boolean field and it will be true for which teachers are the current course coordinators for a specific course... additionally, maybe if the teacher has a whatsapp or phone number to add that and/or if they only teach on one campus such as lev or tal ...