Skip to content
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

create single instructor badge #1162

Open
ErinBecker opened this issue Mar 22, 2017 · 10 comments

Comments

Projects
None yet
5 participants
@ErinBecker
Copy link
Contributor

commented Mar 22, 2017

The Trainer community and staff have decided to move to a single checkout procedure that qualifies the new instructor as both a Data and Software Carpentry instructor. We need to implement a new "instructor" badge and automatically give it to any existing person who has either a Data or Software carpentry instructor badge (or both).

@chrismedrela

This comment has been minimized.

Copy link
Collaborator

commented Mar 22, 2017

@ErinBecker so what are the requirements to become certified instructor? Is it still passing TTT training, homework, discussion session and demo session, just without any distinction between SWC and DC?

@maneesha there are two approaches:

  • keep "SWC Instr." and "DC Instr." badges and add "Instructor" badge;
  • reuse "SWC Instructor" and rename it to "Instructor", award it to all people who have "DC Instructor" badge and delete "DC Instructor" badge.

The last one is simpler to implement, but results in some information loss. Any opinions?

Note to person implementing it: this requires the following changes:

  • write migration that adds Instructor badge to all people who have SWC or DC Instructor badge
  • add Instructor badge
  • trainees list view: eligible column, change the algorithm compucing eligibility
  • trainees list view: "is swc/dc instructor?" filter
  • people list view: dc?, swc? columns
  • changes to requirement types: add "Homework" and "Demo Session"
  • find workshops stuff: instructor badges filter and column

And probably more changes.

@ErinBecker

This comment has been minimized.

Copy link
Contributor Author

commented Mar 22, 2017

@chrismedrela Thanks for the speedy response! The second option (reuse SWC instructor and rename to Instructor) is my preference.

The requirements are as you listed "passing TTT training, homework, discussion session and demo session, just without any distinction between SWC and DC?"

@pbanaszkiewicz

This comment has been minimized.

Copy link
Contributor

commented Mar 25, 2017

@chrismedrela how big this change looks? Pretty-big, medium, small?

@chrismedrela

This comment has been minimized.

Copy link
Collaborator

commented Mar 27, 2017

@pbanaszkiewicz looks like small changes in a lot of places, so I'd say medium.

@maneesha

This comment has been minimized.

Copy link
Contributor

commented Mar 28, 2017

@ErinBecker in the interest of data persistency I'd recommend creating a new badge rather than reusing SWC instructor.

@maneesha

This comment has been minimized.

Copy link
Contributor

commented Apr 4, 2017

Also note that this will require changing the Instructor training progress to not have separate DC Demo, SWC Demo and DC homework, SWC homework.

@pbanaszkiewicz

This comment has been minimized.

Copy link
Contributor

commented Aug 11, 2018

Ref #1232.

The instructor training progress requirements have been changed (#1226 and corresponding PR #1296).

@maneesha

This comment has been minimized.

Copy link
Contributor

commented Jun 25, 2019

I know we've brought this up with our Trainers community, but I don't think we are ready to take action. Is that right? In that case I'd suggest option 3 for now. Let me know what you think @karenword @ErinBecker

Which of the following applies to when you want to see this fixed?

  1. Would like to see this fixed as soon as possible, in the next 1-2 development cycles (i.e., by September 2019)
  2. Would like to see this fixed later (by the end of 2019)
  3. Would like to see this fixed, but possibly as part of a larger project that needs more planning
  4. Not sure
  5. No longer relevant - OK to close
@karenword

This comment has been minimized.

Copy link
Collaborator

commented Jun 25, 2019

@ErinBecker

This comment has been minimized.

Copy link
Contributor Author

commented Jul 5, 2019

Agreed with Karen - 3. this will require some conversation with the community to make sure that we're going the right direction. For example, whether we're doing ONLY a single "The Carpentries" badge or if we're doing "The Carpentries" + optional lesson specific badges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.