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

Add SimpleTeam and SimpleMembership models #46

Merged
merged 15 commits into from
Jul 3, 2016
Merged

Add SimpleTeam and SimpleMembership models #46

merged 15 commits into from
Jul 3, 2016

Conversation

jacobwegner
Copy link
Contributor

@jacobwegner jacobwegner commented Mar 14, 2016

resolves #25.

Provides:

  • Abstract classes BaseTeam and BaseMembership
  • Concrete classes SimpleTeam and SimpleMembership
  • Changes existing Team and Membership classes to inherit from BaseTeam and BaseMembership.

The SimpleTeam and SimpleMembership models are designed to be paired with domain-specific models (for example a team consisting of instructors for a given course, and a second team made up of students for a course in a learning management system), while the "default" Team and Membership models continue to serve organizing individuals into teams across an entire site.

Remaining:

  • Squash and rename migrations into a single migration

@coveralls
Copy link

coveralls commented Jul 3, 2016

Coverage Status

Coverage increased (+1.7%) to 49.928% when pulling 5475ea5 on issue-25 into 7c3bd73 on master.

@jacobwegner jacobwegner merged commit 655c884 into master Jul 3, 2016
@jacobwegner jacobwegner deleted the issue-25 branch July 3, 2016 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split Team model functionality to abstract and concrete classes
3 participants