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

Allow multiple memberships between user and group #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jbx26
Copy link
Collaborator

@jbx26 jbx26 commented Mar 4, 2014

Wenn ein Benutzer von einer Gruppe in eine andere wechselt, wird er aus der anderen Gruppe nicht entfernt, sondern lediglich die Gültigkeit wird invalidiert. Wenn er dann später wieder Mitglied in dieser Gruppe ist, dann ist er zweimal Mitglied in derselben Gruppe. Jede Mitgliedschaft hat aber unterschiedliche Gültigkeit.

Dazu habe ich

  1. Specs geschrieben, die den Promotion-Workflow und die Bricks testen
  2. Die Callbacks am User zum Anlegen eines Accounts und zum Zuordnen zu einer Gruppe habe ich entfernt und stattdessen die save-Methode überschrieben.
  3. Das Akronym ugm durch membership ersetzt
  4. Bei diversen Specs Wartezeiten eingebaut
  5. Invalidieren einer UserGroupMembership erfolgt eine Sekunde vorher, damit sie jetzt bereits ungütlig ist
  6. child_users<< ersetzt
  7. Das Email-Format berichtigt

@fiedl fiedl force-pushed the master branch 2 times, most recently from b4150a4 to 1be2070 Compare September 26, 2014 20:07
@fiedl fiedl force-pushed the master branch 2 times, most recently from 3e70e6d to af086ae Compare September 30, 2014 10:25
@fiedl fiedl force-pushed the master branch 5 times, most recently from 4478ccc to 39728aa Compare October 25, 2014 19: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.

1 participant