DGNum/gestioCOF
14
2
Fork 1
Code Issues 78 Pull requests 14 Projects Releases 12 Packages Wiki Activity

WIP: Les permissions du CMS peuvent être attribuées aux groupes K-Fêt (depuis le site K-Fêt) + quelques améliorations #565

Draft
delobell wants to merge 17 commits from aureplop/kfet-auth_cms into aureplop/kfet-auth
pull from: aureplop/kfet-auth_cms
merge into: DGNum:aureplop/kfet-auth
Conversation 16 Commits 17 Files changed 50 +1814 -411
delobell commented 2017-10-16 13:49:24 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Ajout/modifs

  • Les permissions inutilisées sont supprimées et ne polluent donc plus le formulaire.

Les permissions créées lors d'un précédant appel à migrate sont supprimées dans des migrations des apps kfet et kfetauth.
Quelques renommages (name de kfet.is_team et codename de kfet.view_negs) ont lieu (dans une migration kfet).
La permission kfet.manage_perms est supprimée en faveur des permissions kfetauth.(view|add|change)_group. Les groupes possédant l'ancienne permission obtiennent les 3 nouvelles (via une migration kfetauth).

  • Les permissions apparaissent groupées par content_type avec le champ de formulaire BasePermissionsField.

  • Il est possible d'étendre ce qu'affiche/gère ce formulaire en définissant des "extras" (voir kfet/cms/views.py pour un exemple).

  • Les permissions du CMS peuvent dorénavant être attribuées depuis ce formulaire.

Cela comprend les permissions relatives aux pages, images, documents et à l'accès au site d'admin de Wagtail.
Seules les permissions des pages /k-fet/ et de ses descendantes ainsi que celles concernant la collection K-Fêt et ses descendantes peuvent être données.
Pour ce faire, les formsets/forms adéquats de Wagtail sont customisés (via les fonctions définies dans kfet/cms/forms.py).

Misc

  • "Décalage" de la class form-horizontal dans le template kfet/base_form.html vers les templates "fils". Cela posait problème au form-inline présents dans la balise <form>.
  • kfet/form_submit_snippet.html est doté d'une valeur par défaut ("Enregistrer") si value n'est pas accessible depuis le template.
  • Ajout d'une dépendance : fork de georgema1982 de django-formset-js (par tim_heap). Cette dépendance facilite la manipulation des formsets. Le fork permet de restaurer un formulaire marqué comme supprimé.
  • Pour la review, il peut être plus pratique de review commit par commit.
  • Fixes #168.
## Ajout/modifs - Les permissions inutilisées sont supprimées et ne polluent donc plus le formulaire. Les permissions créées lors d'un précédant appel à `migrate` sont supprimées dans des migrations des apps `kfet` et `kfetauth`. Quelques renommages (`name` de `kfet.is_team` et `codename` de `kfet.view_negs`) ont lieu (dans une migration `kfet`). La permission `kfet.manage_perms` est supprimée en faveur des permissions `kfetauth.(view|add|change)_group`. Les groupes possédant l'ancienne permission obtiennent les 3 nouvelles (via une migration `kfetauth`). - Les permissions apparaissent groupées par `content_type` avec le champ de formulaire `BasePermissionsField`. - Il est possible d'étendre ce qu'affiche/gère ce formulaire en définissant des "extras" (voir `kfet/cms/views.py` pour un exemple). - Les permissions du CMS peuvent dorénavant être attribuées depuis ce formulaire. Cela comprend les permissions relatives aux pages, images, documents et à l'accès au site d'admin de Wagtail. Seules les permissions des pages `/k-fet/` et de ses descendantes ainsi que celles concernant la collection K-Fêt et ses descendantes peuvent être données. Pour ce faire, les formsets/forms adéquats de Wagtail sont customisés (via les fonctions définies dans `kfet/cms/forms.py`). ## Misc - "Décalage" de la class `form-horizontal` dans le template `kfet/base_form.html` vers les templates "fils". Cela posait problème au `form-inline` présents dans la balise `<form>`. - `kfet/form_submit_snippet.html` est doté d'une valeur par défaut ("Enregistrer") si `value` n'est pas accessible depuis le template. - Ajout d'une dépendance : [fork de georgema1982](https://bitbucket.org/georgema1982/django-formset-js) de [django-formset-js (par tim_heap)](https://bitbucket.org/tim_heap/django-formset-js). Cette dépendance facilite la manipulation des formsets. Le fork permet de restaurer un formulaire marqué comme supprimé. - Pour la review, il peut être plus pratique de review commit par commit. - Fixes #168.
delobell commented 2017-10-17 16:51:30 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 5 commits

  • 5502c687 - Clean permissions objects
  • 82582866 - Clean forms/views/urls related to kfetauth.Group…
  • 07f1a535 - CMS permissions can be managed from group views.
  • 2c76bea1 - Better display of objects of BasePermissionsField
  • 097ee441 - Organize migrations to avoid issues with…

Compare with previous version

added 5 commits * 5502c687 - Clean permissions objects * 82582866 - Clean forms&#x2F;views&#x2F;urls related to kfetauth.Group… * 07f1a535 - CMS permissions can be managed from group views. * 2c76bea1 - Better display of objects of BasePermissionsField * 097ee441 - Organize migrations to avoid issues with… [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/261/diffs?diff_id=677&start_sha=048b889b68d6b347d5cea1b01689478ff4a4534d)
delobell commented 2017-10-17 16:52:20 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

Modifications effectuées

  • Wrap des non_form_errors des formsets des permissions (pages, docs, images) par <span class="help-block">.
  • Correction de bug: les champs permissions de GroupForm et SnippetsCmsGroupForm ayant le même nom, cela donnait un problème de validation. La correction consiste à ajouter des prefix aux forms. Ceci est fait dans la définition des extras plutôt que sur les forms eux-mêmes car seulement Django >= 1.9 autorise de donner l'attribut prefix directement à une classe de formulaire 😞
  • Correction de bug: le template kfet/account.html vérifie les bons labels de permissions.
  • Réorganisation des migrations pour éviter des problèmes si les migrations de master ont déjà été appliquées.
## Modifications effectuées - Wrap des `non_form_errors` des formsets des permissions (pages, docs, images) par `<span class="help-block">`. - Correction de bug: les champs `permissions` de `GroupForm` et `SnippetsCmsGroupForm` ayant le même nom, cela donnait un problème de validation. La correction consiste à ajouter des `prefix` aux forms. Ceci est fait dans la définition des extras plutôt que sur les forms eux-mêmes car seulement Django >= 1.9 autorise de donner l'attribut `prefix` directement à une classe de formulaire :disappointed: - Correction de bug: le template `kfet/account.html` vérifie les bons labels de permissions. - Réorganisation des migrations pour éviter des problèmes si les migrations de `master` ont déjà été appliquées.
delobell commented 2017-10-17 17:00:31 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

La réorganisation permet de tester plus facilement l'application des migrations à partir de master:

$ rm db.sqlite3
$ git co master
$ bash provisioning/prepare_django.sh
# Modification possible ici des groupes créés depuis les données de développement (via shell, runserver…)
$ git co aureplop/kfet-auth_cms
$ python manage.py migrate

Les modifications observables sont:

  • le renommage de Is member of the team en Membre de l'équipe,
  • la suppression des permissions non utilisées,
  • les objets qui avaient la permission view_negs "Voir ~(les négatifs K-Fêt)" ont la permission kfetauth.view_accountnegative (simple renommage du codename).
  • les groupes qui avaient la permission manage_perms "Gérer les permissions K-Fêt" obtiennent les permissions kfetauth.{view,add,change}_group.
La réorganisation permet de tester plus facilement l'application des migrations à partir de master: ```bash $ rm db.sqlite3 $ git co master $ bash provisioning/prepare_django.sh # Modification possible ici des groupes créés depuis les données de développement (via shell, runserver…) $ git co aureplop/kfet-auth_cms $ python manage.py migrate ``` Les modifications observables sont: - le renommage de `Is member of the team` en `Membre de l'équipe`, - la suppression des permissions non utilisées, - les objets qui avaient la permission `view_negs "Voir ~(les négatifs K-Fêt)"` ont la permission `kfetauth.view_accountnegative` (simple renommage du `codename`). - les groupes qui avaient la permission `manage_perms "Gérer les permissions K-Fêt"` obtiennent les permissions `kfetauth.{view,add,change}_group`.
delobell commented 2017-10-24 19:50:23 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 51 commits

  • 097ee441...09290131 - 47 commits from branch aureplop/kfet-auth
  • c524da22 - Merge branch 'aureplop/kfet-auth' into aureplop/kfet-auth_cms
  • aa405d21 - Fix tests…
  • 03a0c588 - Merge branch 'aureplop/kfet-auth' into aureplop/kfet-auth_cms
  • e39f8899 - Fix migration history

Compare with previous version

added 51 commits * 097ee441...09290131 - 47 commits from branch `aureplop/kfet-auth` * c524da22 - Merge branch &#x27;aureplop&#x2F;kfet-auth&#x27; into aureplop&#x2F;kfet-auth_cms * aa405d21 - Fix tests… * 03a0c588 - Merge branch &#x27;aureplop&#x2F;kfet-auth&#x27; into aureplop&#x2F;kfet-auth_cms * e39f8899 - Fix migration history [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/261/diffs?diff_id=686&start_sha=097ee44131f5a511b6aea2b3bc578c700a2f9588)
delobell commented 2017-10-24 19:50:25 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

marked as a Work In Progress

marked as a **Work In Progress**
delobell commented 2017-10-24 19:50:25 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

changed the description

changed the description
delobell commented 2017-10-24 19:51:16 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

WIP, TODO: fix et ajout de tests

WIP, TODO: fix et ajout de tests
delobell commented 2017-10-25 13:56:39 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 1 commit

  • 94f21c06 - SnippetCmsGroupForm should not clean previous permissions

Compare with previous version

added 1 commit * 94f21c06 - SnippetCmsGroupForm should not clean previous permissions [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/261/diffs?diff_id=687&start_sha=e39f88991ec369f20031332a542d718d87f35f34)
delobell commented 2017-10-25 20:30:48 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 6 commits

  • 94f21c06...13f01020 - 3 commits from branch aureplop/kfet-auth
  • 94ab754f - Move group-related views tests to kfetauth tests
  • c5af9bdc - Fix SnippetsCmsForm
  • e7060ea8 - Merge branch 'aureplop/kfet-auth' into aureplop/kfet-auth_cms

Compare with previous version

added 6 commits * 94f21c06...13f01020 - 3 commits from branch `aureplop/kfet-auth` * 94ab754f - Move group-related views tests to kfetauth tests * c5af9bdc - Fix SnippetsCmsForm * e7060ea8 - Merge branch &#x27;aureplop&#x2F;kfet-auth&#x27; into aureplop&#x2F;kfet-auth_cms [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/261/diffs?diff_id=691&start_sha=94f21c062a5ec1f602109f448ee3385df07fa7b6)
delobell commented 2017-10-25 20:48:24 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 3 commits

  • a0895962 - Fix SnippetsCmsForm
  • 63a79d2b - Fix tests, force reevaluate field queryset
  • 6ec7a615 - typos

Compare with previous version

added 3 commits * a0895962 - Fix SnippetsCmsForm * 63a79d2b - Fix tests, force reevaluate field queryset * 6ec7a615 - typos [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/261/diffs?diff_id=692&start_sha=e7060ea8a4a1ce5eb2e2e32bdb72c1bae7c0d82f)
delobell commented 2017-10-25 20:57:38 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

added 3 commits

Compare with previous version

added 3 commits * a7dbc64e - Fix SnippetsCmsForm * 6d90311a - Fix tests, force reevaluate field queryset * 02f1e834 - typos [Compare with previous version](https://git.eleves.ens.fr/cof-geek/gestioCOF/merge_requests/261/diffs?diff_id=693&start_sha=6ec7a615309a7cd80823579e0ec5eb2cd243314c)
delobell commented 2018-09-30 15:03:47 +02:00 (Migrated from git.eleves.ens.fr)
Copy link

@mpepin Si on a du temps après !274, on essaie de se la faire ?

@mpepin Si on a du temps après !274, on essaie de se la faire ?
lstephan commented 2019-01-14 21:13:32 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

C'est toujours un WIP ? C'est possible (pour avoir plus de tests) de merge master dessus ?

C'est toujours un WIP ? C'est possible (pour avoir plus de tests) de merge `master` dessus ?
delobell commented 2019-01-14 22:06:06 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

J'sais pas si c'est en WIP. Je me demande pourquoi la MR target aureplop/kfet-auth. Je vois plus ce qui a sur cette branche…
Jusqu'ici tout va bien, ça sera la surprise.

master incoming

J'sais pas si c'est en WIP. Je me demande pourquoi la MR target aureplop/kfet-auth. Je vois plus ce qui a sur cette branche… Jusqu'ici tout va bien, ça sera la surprise. master incoming
delobell commented 2019-01-14 22:08:09 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

added 4 commits

  • 02f1e834...13f01020 - 3 commits from branch aureplop/kfet-auth
  • e3e86085 - Merge branch 'aureplop/kfet-auth' into aureplop/kfet-auth_cms

Compare with previous version

added 4 commits <ul><li>02f1e834...13f01020 - 3 commits from branch <code>aureplop/kfet-auth</code></li><li>e3e86085 - Merge branch &#39;aureplop/kfet-auth&#39; into aureplop/kfet-auth_cms</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/261/diffs?diff_id=1082&start_sha=02f1e834208aba80ce126deade0c5937f5d1e813)
delobell commented 2019-01-14 22:13:16 +01:00 (Migrated from git.eleves.ens.fr)
Copy link

mentioned in merge request !345

mentioned in merge request !345
This pull request has changes conflicting with the target branch.
  • cof/settings/common.py
  • kfet/apps.py
  • kfet/auth/fields.py
  • kfet/auth/forms.py
  • kfet/auth/migrations/0001_initial.py
  • kfet/auth/migrations/0003_existing_groups.py
  • kfet/auth/tests.py
  • kfet/auth/views.py
  • kfet/cms/management/commands/kfet_loadwagtail.py
  • kfet/config.py
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin aureplop/kfet-auth_cms:aureplop/kfet-auth_cms
git checkout aureplop/kfet-auth_cms

Merge

Merge the changes and update on Forgejo.
git checkout aureplop/kfet-auth
git merge --no-ff aureplop/kfet-auth_cms
git checkout aureplop/kfet-auth
git merge --ff-only aureplop/kfet-auth_cms
git checkout aureplop/kfet-auth_cms
git rebase aureplop/kfet-auth
git checkout aureplop/kfet-auth
git merge --no-ff aureplop/kfet-auth_cms
git checkout aureplop/kfet-auth
git merge --squash aureplop/kfet-auth_cms
git checkout aureplop/kfet-auth
git merge --ff-only aureplop/kfet-auth_cms
git checkout aureplop/kfet-auth
git merge aureplop/kfet-auth_cms
git push origin aureplop/kfet-auth
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é

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
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#565
No description provided.
Delete branch "aureplop/kfet-auth_cms"

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?