-
-
Notifications
You must be signed in to change notification settings - Fork 171
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Should we remove inactive instructors from the team page? #280
Comments
I think so, do we have an "inactive" query cooked up? It would be nice to have a lightweight integration that just flips the bios to unpublished rather than deleting them. Maybe we build the list of active instructors weekly or monthly from AMY, similar to how we build the calendar from AMY. |
I agree we should hide inactive instructors. We should agree on how we're defining "inactive" -- it needs to be a function of when they were certified & how often they've taught. Members are defined by having taught at least 2x in the past 730 days or having made a significant contribution to the work of SWC so we'd want to count those too. |
On Wed, Mar 02, 2016 at 07:19:22AM -0800, Jonah Duckles wrote:
Is this so the inactive list could be automatically pulled from AMY, |
@maneesha, shouldn't it just be the inverse of "members", if you're no longer qualified as a member you're inactive? |
@jduckles -- I would consider "active instructors" to be bigger than members. Members have taught at least 2x in the past 730 days. I think "active instructors" should include newly trained instructors who just haven't had a chance yet to teach, or maybe have only taught once. That's why I think it should be a function of how often they've taught and when they got certified. I'm not sure what rules should apply there though. So "inactive" is not just someone who's never taught -- it's someone who's never taught and was certified more than X months ago. |
Ok, I agree, it is a union of the sets "members" and "recently badged". For some definition of recent, I'm inclined to use the same definition we're using for membership, I think it was two years. |
We now have a discussion going on about this in the board's repo. |
No description provided.
The text was updated successfully, but these errors were encountered: