DGNum/gestioCOF
15
2
Fork 2
Code Issues 79 Pull requests 19 Projects Releases 12 Packages Wiki Activity Actions

Des groupes et permissions pour la K-Fêt #649

Closed
delobell wants to merge 11 commits from aureplop/kfet-auth into master
pull from: aureplop/kfet-auth
merge into: DGNum:master
Conversation 24 Commits 11 Files changed 15 +490 -150
delobell commented 2019-01-14 22:13:15 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

@lstephan Est-ce qu'on enverrait pas ça dans master ? Je vais review ça à un moment.
Y'a au moins un truc pas normal : la migration kfetauth est modifiée 🤔

On modifiera la target de !261 ensuite.

J'en profite pour voir ce que la CI va cracher.

@lstephan Est-ce qu'on enverrait pas ça dans master ? Je vais review ça à un moment. Y'a au moins un truc pas normal : la migration kfetauth est modifiée :thinking: On modifiera la target de !261 ensuite. J'en profite pour voir ce que la CI va cracher.
delobell commented 2019-01-14 22:24:30 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed title from K-Fêt to {+Module d'auth +}K-Fêt

changed title from **K-Fêt** to **{+Module d'auth +}K-Fêt**
delobell commented 2019-01-14 22:24:30 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
lstephan commented 2019-01-14 22:30:28 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Pourquoi c'est pas un proxy lui ?

Pourquoi c'est pas un `proxy` lui ?
delobell commented 2019-01-14 22:42:00 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 308 commits

  • 13f01020...2140a597 - 307 commits from branch master
  • fcf4a257 - Merge branch 'master' into aureplop/kfet-auth

Compare with previous version

added 308 commits <ul><li>13f01020...2140a597 - 307 commits from branch <code>master</code></li><li>fcf4a257 - Merge branch &#39;master&#39; into aureplop/kfet-auth</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/345/diffs?diff_id=1085&start_sha=13f01020f7ec2eef9077ab235cfed96427e3c2a7)
delobell commented 2019-01-14 22:44:27 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

@lstephan j'aurais bien dit oui, mais comment je fais pour identifier les groupes K-Fêt alors ?
Et on a pas accès au Group de Django pour ajouter un field ou un M2M :(

@lstephan j'aurais bien dit oui, mais comment je fais pour identifier les groupes K-Fêt alors ? Et on a pas accès au `Group` de Django pour ajouter un field ou un M2M :(
delobell commented 2019-01-14 22:46:16 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

En principe, master a été merge dans cette branche. Le coup de linter est passé, les migrations ont été décalées.
Je regarde dans la semaine.

En principe, master a été merge dans cette branche. Le coup de linter est passé, les migrations ont été décalées. Je regarde dans la semaine.
delobell commented 2019-01-14 22:48:37 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Suis plus que tenté de renommer Group en KFetGroup. Ça serait sûrement plus pratique à utiliser.

Suis plus que tenté de renommer `Group` en `KFetGroup`. Ça serait sûrement plus pratique à utiliser.
delobell commented 2019-01-14 22:49:00 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Et du coup, KFetPermission

Et du coup, `KFetPermission`
delobell commented 2019-01-14 22:50:17 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

changed title from {-Module d'auth-} K-Fêt to {+Des groupes et permissions pour la+} K-Fêt

changed title from **{-Module d'auth-} K-Fêt** to **{+Des groupes et permissions pour la+} K-Fêt**
lstephan commented 2019-01-14 22:55:13 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Mais là, il n'a pas de M2M ni de field supplémentaire...

Mais là, il n'a pas de M2M ni de field supplémentaire...
lstephan commented 2019-01-14 22:56:15 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Ça me paraît effectivement plus lisible ^^

Ça me paraît effectivement plus lisible ^^
delobell commented 2019-01-14 23:01:41 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • d06c1561 - kfet -- Fix auth tests

Compare with previous version

added 1 commit <ul><li>d06c1561 - kfet -- Fix auth tests</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/345/diffs?diff_id=1086&start_sha=fcf4a25745bdac10447417c41841eb75ffe33ded)
delobell commented 2019-01-14 23:07:37 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Avec cet héritage y'a un champ group_ptr qui doit s'ajouter, un petit one-to-one vers Group.
Et on peut se sortir du __startswith="K-Fêt -" au passage.
En faisant des choses similaires pour des groupes (à faire) pour le COF et le BdA on peut isoler les schmilblicks.

Avec [cet héritage](https://docs.djangoproject.com/en/2.1/topics/db/models/#multi-table-inheritance) y'a un champ `group_ptr` qui doit s'ajouter, un petit one-to-one vers `Group`. Et on peut se sortir du `__startswith="K-Fêt -"` au passage. En faisant des choses similaires pour des groupes (à faire) pour le COF et le BdA on peut isoler les schmilblicks.
lstephan commented 2019-01-14 23:08:01 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Qu'est ce que fait ce truc ? 😖

Qu'est ce que fait ce truc ? :confounded:
delobell commented 2019-01-14 23:09:29 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 569ff06f - kfet -- Fix auth tests

Compare with previous version

added 1 commit <ul><li>569ff06f - kfet -- Fix auth tests</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/345/diffs?diff_id=1088&start_sha=d06c1561d403b131953d21b52b19ecad0a932de1)
delobell commented 2019-01-14 23:14:01 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Tu te rappelles du moment où on supprimait des permissions COF en passant par la K-Fêt ? Probablement une affaire de groupe de trésorerie.
Le problème c'était qu'un champ M2M de form correspondant à un M2M de modèle fait un .set() lors du save().

J'ai "juste" isolé ce qui était déjà dans les formulaires de UserGroup et Group de l'app kfet.
C'était utilisé à deux endroits…

Sûrement qu'un peu de doc serait la bienvenue ^^

Tu te rappelles du moment où on supprimait des permissions COF en passant par la K-Fêt ? Probablement une affaire de groupe de trésorerie. Le problème c'était qu'un champ M2M de form correspondant à un M2M de modèle fait un `.set()` lors du `save()`. J'ai "juste" isolé ce qui était déjà dans les formulaires de `UserGroup` et `Group` de l'app `kfet`. C'était utilisé à deux endroits… Sûrement qu'un peu de doc serait la bienvenue ^^
delobell commented 2019-01-14 23:17:46 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

Compare with previous version

added 1 commit <ul><li>84bbe78e - kfet -- Fix auth tests</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/345/diffs?diff_id=1092&start_sha=569ff06fe693c2f98e18a58efec5430535fb6b30)
delobell commented 2019-01-14 23:28:46 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Prefetch permissions, User ?

Prefetch `permissions`, `User` ?
delobell commented 2019-01-14 23:35:19 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Faudrait faire que KFet.objects soit kfetcore (pour le moment en tout cas).
Un KFetGroup.objects.all().delete() serait regrété au-delà de la K-Fêt…

Faudrait faire que `KFet.objects` soit `kfetcore` (pour le moment en tout cas). Un `KFetGroup.objects.all().delete()` serait regrété au-delà de la K-Fêt…
delobell commented 2019-01-14 23:36:22 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Faut se détendre un peu

Faut se détendre un peu
lstephan commented 2019-02-04 22:58:30 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Pourquoi ne pas override clean, voire même, avec un peu plus de machinerie, clean_<field> ? Ça évite de réécrire la fonction save_m2m (et est donc plus robuste si elle change), on a accès à tout ce qu'il faut dans clean, et c'est plus lisible ^^

Pourquoi ne pas override `clean`, voire même, avec un peu plus de machinerie, `clean_<field>` ? Ça évite de réécrire la fonction `save_m2m` (et est donc plus robuste si elle change), on a accès à tout ce qu'il faut dans `clean`, et c'est plus lisible ^^
lstephan commented 2020-01-05 17:28:24 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

Des choses intéressantes dans cette MR, mais trop de retard sur master... Je vais voir ce que je peux récupérer sur une autre branche.

Des choses intéressantes dans cette MR, mais trop de retard sur master... Je vais voir ce que je peux récupérer sur une autre branche.
lstephan commented 2020-08-03 19:13:07 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !438

mentioned in merge request !438
mpepin commented 2020-09-05 22:13:09 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

closed

closed

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
devtype -- backend

Au programme : du Python, du Django...

  
devtype -- docs

   
devtype -- frontend

   
devtype -- user interface

Au programme : templating, JS, CSS...

  
difficulty -- easy

   
difficulty -- hard

Difficile et/ou risqué

  
difficulty -- normal

   
Doing

   
domain -- bda

Concerne l'application BdA

  
domain -- bds

   
domain -- cof

Concerne l'application COF

  
domain -- core

Dépendances externes, CI, linting...

  
domain -- kfet

Concerne l'application K-Fêt

  
Good first issue

   
priority -- high

Urgent quoi

  
priority -- low

   
priority -- medium

   
priority -- staff-wanted

L'équipe dirigeante du domaine en question souhaite cette fonctionnalité.

  
status -- development

En cours de développement

  
status -- discussion

Avant de coder, c'est bien de discuter

  
status -- need review

En attente de code review

  
status -- production

L'ajout/la modification est présente en production

  
status -- ready to merge

Plus qu'à merger !

  
status -- todo

Le développement devrait bientôt commencer

  
To Do

   
type -- bug

Signalement ou correction de bug

  
type -- hygiene

Tout ce qui amène à du code plus propre et un repo mieux organisé.

  
type -- improvement

Ajout/Demande d'évolution mineure d'une fonctionnalité existante

  
type -- new feature

Ajout/Demande d'une nouvelle fonctionnalité

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
devtype -- backend
devtype -- docs
devtype -- frontend
devtype -- user interface
difficulty -- easy
difficulty -- hard
difficulty -- normal
Doing
domain -- bda
domain -- bds
domain -- cof
domain -- core
domain -- kfet
Good first issue
priority -- high
priority -- low
priority -- medium
priority -- staff-wanted
status -- development
status -- discussion
status -- need review
status -- production
status -- ready to merge
status -- todo
To Do
type -- bug
type -- hygiene
type -- improvement
type -- new feature
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/gestioCOF#649
No description provided.
Delete branch "aureplop/kfet-auth"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?