search

github-education-resources / teachers_pet

Command line tool to help teachers use GitHub in their classrooms
https://education.github.com/guide
MIT License
187 stars 74 forks source link

Split batch creation of teams from general adding to a team #68

Closed afeld closed 10 years ago

afeld commented 10 years ago

create_teams was doing too much: it was handing creation of teams as

  1. Creating teams for groups of students (e.g. for group projects)
  2. Creating one team per student for individual assignments
  3. Adding instructors to the Owners team

This PR splits 1+2 from 3, and makes 3 more generic, in that it can add a list of users to any team. Also, created an API wrapper to bring the common logic out of the Action classes.

afeld commented 10 years ago

@thedaniel @rick @ymendel P.S. The new commands/class names here (AddToTeam and CreateStudentTeams) were what I was looking for feedback on yesterday.

rick commented 10 years ago

@afeld -- ok, :cool: -- I dig it. :+1: