DGNum/gestioCOF

Fork 2

Change le fonctionnement du gel de compte #797

Merged

lstephan merged 0 commits from Aufinal/let_it_go into master

2021-06-15 17:24:44 +02:00

lstephan commented

2021-02-24 00:34:53 +01:00

(Migrated from git.eleves.ens.fr)

Seules les personnes avec la permission manage_perms peuvent maintenant (dé)geler un compte
en conséquence, tous les comptes sont dégelés via migration,
on ne peut plus override le gel pour compter des consos/transferts sur un compte gelé,
les comptes gelés n'ont plus accès à K-Psul.

Fixes #280

Builds from !490

- Seules les personnes avec la permission `manage_perms` peuvent maintenant (dé)geler un compte - en conséquence, tous les comptes sont dégelés via migration, - on ne peut plus override le gel pour compter des consos/transferts sur un compte gelé, - les comptes gelés n'ont plus accès à K-Psul. Fixes #280 Builds from !490

lstephan commented

2021-02-24 00:35:04 +01:00

(Migrated from git.eleves.ens.fr)

changed the description

lstephan commented

2021-02-28 02:13:43 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in merge request !494

thubrecht commented

2021-03-17 08:34:32 +01:00

(Migrated from git.eleves.ens.fr)

On peut geler son propre compte, c'est normal ?

lstephan commented

2021-03-17 10:30:27 +01:00

(Migrated from git.eleves.ens.fr)

Depuis un compte qui n'a pas la permission manage_perms ?

Depuis un compte qui n'a pas la permission `manage_perms` ?

thubrecht commented

2021-03-17 10:35:20 +01:00

(Migrated from git.eleves.ens.fr)

Oui

thubrecht commented

2021-03-17 14:24:21 +01:00

(Migrated from git.eleves.ens.fr)

Je crois avoir trouvé, dans AccountForm t'as gardé le champ is_frozen, du coup ça override le AccountFrozenForm

Je crois avoir trouvé, dans `AccountForm` t'as gardé le champ `is_frozen`, du coup ça override le `AccountFrozenForm`

thubrecht commented

2021-03-17 14:26:47 +01:00

(Migrated from git.eleves.ens.fr)

D'ailleurs tu passes pas frozen_form dans le contexte pour render le template

D'ailleurs tu passes pas `frozen_form` dans le contexte pour render le template

lstephan commented

2021-03-17 14:28:19 +01:00

(Migrated from git.eleves.ens.fr)

Oups, j'ai du merder un rebase quelconque !

lstephan commented

2021-03-17 21:02:03 +01:00

(Migrated from git.eleves.ens.fr)

added 1 commit

6786ef15 - Fix frozen forms

Compare with previous version

added 1 commit <ul><li>6786ef15 - Fix frozen forms</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/493/diffs?diff_id=2502&start_sha=3ed6ca4956ea377f3d6670af5701b8f89a9a5f35)

thubrecht commented

2021-03-17 21:39:36 +01:00

(Migrated from git.eleves.ens.fr)

Il faudrait mettre un warning quand on essaie de modifier le gel de compte

else:
    if group_form.has_changed():
        warnings.append("statut d'équipe")
    if frozen_form.has_changed():
        warnings.append("gel de compte")

Il faudrait mettre un warning quand on essaie de modifier le gel de compte ```python else: if group_form.has_changed(): warnings.append("statut d'équipe") if frozen_form.has_changed(): warnings.append("gel de compte") ```

lstephan commented

2021-06-15 14:07:50 +02:00

(Migrated from git.eleves.ens.fr)

added 41 commits

6786ef15...0351f672 - 32 commits from branch master
99809209 - Change les permissions pour geler/dégeler un compte
4136cb68 - Unfreeze every account
1e44550e - New frozen function
63738e8e - Frozen error display
93d283fe - Remove unused permission
a947b9d3 - Fix decorator
16dee0c1 - Remove print
b9aaf6a1 - Fix test
7bf0c5f0 - Fix frozen forms

Compare with previous version

added 41 commits <ul><li>6786ef15...0351f672 - 32 commits from branch <code>master</code></li><li>99809209 - Change les permissions pour geler/dégeler un compte</li><li>4136cb68 - Unfreeze every account</li><li>1e44550e - New frozen function</li><li>63738e8e - Frozen error display</li><li>93d283fe - Remove unused permission</li><li>a947b9d3 - Fix decorator</li><li>16dee0c1 - Remove print</li><li>b9aaf6a1 - Fix test</li><li>7bf0c5f0 - Fix frozen forms</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/493/diffs?diff_id=2967&start_sha=6786ef150fca37c478a7171330b5f41468b6ce0b)

thubrecht commented

2021-06-15 14:46:40 +02:00

(Migrated from git.eleves.ens.fr)

L'indentation est pas la même que le case 403 😇

L'indentation est pas la même que le `case 403` :innocent:

lstephan commented

2021-06-15 14:48:39 +02:00

(Migrated from git.eleves.ens.fr)

changed this line in version 6 of the diff

changed this line in [version 6 of the diff](/klub-dev-ens/gestioCOF/-/merge_requests/493/diffs?diff_id=2970&start_sha=7bf0c5f09e13b58dc8d474d4294b2589b309c424#c7e6f7936f7a86bfec74333d6499658b031b27b5_124_124)

lstephan commented

2021-06-15 14:48:40 +02:00

(Migrated from git.eleves.ens.fr)

added 1 commit

02584982 - gnagnagna

Compare with previous version

added 1 commit <ul><li>02584982 - gnagnagna</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/493/diffs?diff_id=2970&start_sha=7bf0c5f09e13b58dc8d474d4294b2589b309c424)

thubrecht commented

2021-06-15 15:08:01 +02:00

(Migrated from git.eleves.ens.fr)

C'est normal de pouvoir supprimer son propre compte quand il est gelé ?

lstephan commented

2021-06-15 15:11:49 +02:00

(Migrated from git.eleves.ens.fr)

C'est complètement unintended :upside_down:

thubrecht commented

2021-06-15 15:17:35 +02:00

(Migrated from git.eleves.ens.fr)

En fait, même en étant gelé tu peux supprimer n'importe quel compte

lstephan commented

2021-06-15 15:21:07 +02:00

(Migrated from git.eleves.ens.fr)

Oui, effectivement : en fait, override has_perm est ultra relou, donc j'ai juste override le décorateur teamkfet_required. Mais certains cas n'utilisent pas du tout ce décorateur pour checker les permissions...

Oui, effectivement : en fait, override `has_perm` est ultra relou, donc j'ai juste override le décorateur `teamkfet_required`. Mais certains cas n'utilisent pas du tout ce décorateur pour checker les permissions...

thubrecht commented

2021-06-15 15:26:41 +02:00

(Migrated from git.eleves.ens.fr)

En fait les endroits où tu regardes has_perm("kfet.is_team"), il faut remplacer par kfet_is_team qui est définie dans decorators.py

En fait les endroits où tu regardes `has_perm("kfet.is_team")`, il faut remplacer par `kfet_is_team` qui est définie dans `decorators.py`

lstephan commented

2021-06-15 15:27:25 +02:00

(Migrated from git.eleves.ens.fr)

J'ai une idée alternative plus clean, je vais tester ça avant =)

lstephan commented

2021-06-15 16:53:03 +02:00

(Migrated from git.eleves.ens.fr)

added 2 commits

a34b83c2 - Use backend to enforce frozen accounts
6a111395 - Fix tests

Compare with previous version

added 2 commits <ul><li>a34b83c2 - Use backend to enforce frozen accounts</li><li>6a111395 - Fix tests</li></ul> [Compare with previous version](/klub-dev-ens/gestioCOF/-/merge_requests/493/diffs?diff_id=2973&start_sha=02584982f6174654d713aea26777fe66d6acde0a)

thubrecht commented

2021-06-15 17:17:34 +02:00

(Migrated from git.eleves.ens.fr)

C'est beaucoup mieux en effet

thubrecht commented

2021-06-15 17:22:10 +02:00

(Migrated from git.eleves.ens.fr)

resolved all threads

thubrecht commented

2021-06-15 17:24:45 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in commit 8743301105