Change le fonctionnement du gel de compte #797
No reviewers
Labels
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
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/gestioCOF#797
Loading…
Reference in a new issue
No description provided.
Delete branch "Aufinal/let_it_go"
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?
manage_perms
peuvent maintenant (dé)geler un compteFixes #280
Builds from !490
changed the description
mentioned in merge request !494
On peut geler son propre compte, c'est normal ?
Depuis un compte qui n'a pas la permission
manage_perms
?Oui
Je crois avoir trouvé, dans
AccountForm
t'as gardé le champis_frozen
, du coup ça override leAccountFrozenForm
D'ailleurs tu passes pas
frozen_form
dans le contexte pour render le templateOups, j'ai du merder un rebase quelconque !
added 1 commit
Compare with previous version
Il faudrait mettre un warning quand on essaie de modifier le gel de compte
added 41 commits
master
99809209
- Change les permissions pour geler/dégeler un compte4136cb68
- Unfreeze every account1e44550e
- New frozen function63738e8e
- Frozen error display93d283fe
- Remove unused permissiona947b9d3
- Fix decorator16dee0c1
- Remove printb9aaf6a1
- Fix test7bf0c5f0
- Fix frozen formsCompare with previous version
L'indentation est pas la même que le
case 403
😇changed this line in version 6 of the diff
added 1 commit
02584982
- gnagnagnaCompare with previous version
C'est normal de pouvoir supprimer son propre compte quand il est gelé ?
C'est complètement unintended :upside_down:
En fait, même en étant gelé tu peux supprimer n'importe quel compte
Oui, effectivement : en fait, override
has_perm
est ultra relou, donc j'ai juste override le décorateurteamkfet_required
. Mais certains cas n'utilisent pas du tout ce décorateur pour checker les permissions...En fait les endroits où tu regardes
has_perm("kfet.is_team")
, il faut remplacer parkfet_is_team
qui est définie dansdecorators.py
J'ai une idée alternative plus clean, je vais tester ça avant =)
added 2 commits
a34b83c2
- Use backend to enforce frozen accounts6a111395
- Fix testsCompare with previous version
C'est beaucoup mieux en effet
resolved all threads
mentioned in commit
8743301105