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

Remove RealGroup and MetaGroup #935

Merged
merged 9 commits into from
Jan 4, 2025
Merged

Remove RealGroup and MetaGroup #935

merged 9 commits into from
Jan 4, 2025

Conversation

imperosol
Copy link
Contributor

@imperosol imperosol commented Nov 30, 2024

En ajoutant automatiquement les utilisateurs aux groupes qui vont bien quand ils rejoignent ou quittent un club, on peut arrêter notre système bizarre où on doit successivement examiner les adhésions aux clubs, puis rechercher à partir de ça un groupe auquel pas vraiment lié ni à l'utilisateur, ni au club, en concaténant le nom du club et "-membres" ou "-bureau".

Pour faire ça, j'ai introduit deux nouvelles colonnes dans la table des clubs :

  • members_group : groupe des membres du club
  • board_group : groupe des membres du bureau du club.

Quand un utilisateur rejoint un club, on l'ajoute au groupe des membres. S'il rejoint le bureau, on l'ajoute au groupe du bureau ET au club des membres. Quand il quitte le club, on le retire des clubs.

Les modèles proxy RealGroup et MetaGroup ont été retirés, mais is_meta n'a pas complètement disparu. Il a été renommé en is_manually_manageable. Ce nouveau champ recouvre une signification un peu plus concrète que la séparation meta/non-meta : il permet de séparer les groupes auxquels on peut assigner ou retirer des membres depuis le site.

A partir du moment où les groupes liés aux clubs deviennent gérables comme le reste, les simplifications viennent en cascade.

Cette PR n'est que la deuxième étape (la première étant #929). Si ça se passe bien, d'autres suivront.

@imperosol imperosol requested a review from klmp200 November 30, 2024 21:53
@imperosol imperosol force-pushed the groups branch 5 times, most recently from 5a0e39b to 15f282a Compare November 30, 2024 22:56
@imperosol imperosol requested a review from NaNoMelo December 1, 2024 10:57
@imperosol imperosol force-pushed the groups branch 2 times, most recently from 5533a15 to 7bab86d Compare December 1, 2024 20:16
@imperosol imperosol force-pushed the groups branch 8 times, most recently from a7dca43 to 2e6554f Compare December 23, 2024 15:16
@imperosol imperosol force-pushed the groups branch 2 times, most recently from 4333fed to b8f851b Compare January 3, 2025 00:18
@imperosol imperosol requested a review from klmp200 January 4, 2025 01:57
Copy link
Contributor

@klmp200 klmp200 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Franchement, c'est plutôt pas mal, à part les points soulignés, j'ai pas vraiment trouvé d'autre problème pour l'instant, c'est cool.

club/tests.py Show resolved Hide resolved
club/models.py Outdated Show resolved Hide resolved
club/models.py Outdated Show resolved Hide resolved
club/models.py Show resolved Hide resolved
@imperosol imperosol requested a review from klmp200 January 4, 2025 15:49
Copy link
Contributor

@klmp200 klmp200 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On continue dans la lignée des pull request terrifiantes

@imperosol imperosol merged commit d08d54b into taiste Jan 4, 2025
3 checks passed
@imperosol imperosol deleted the groups branch January 4, 2025 16:13
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.

2 participants