entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

backoffice: add button to check all permission columns (#66067) #757

Open
fpeters wants to merge 1 commits from wip/66067-admin-permissions-check-all into main
pull from: wip/66067-admin-permissions-check-all
merge into: entrouvert:main
Conversation 1 Commits 1 Files Changed 3 +41 -6
fpeters commented 2023-10-04 11:07:24 +02:00
Owner
Copy Link

En bonus quand tout est coché le bouton décoche tout.

(c'est le genre de petit patchs qui pourrait servir d'entrainement sur des tests js, je pense).

En bonus quand tout est coché le bouton décoche tout. (c'est le genre de petit patchs qui pourrait servir d'entrainement sur des tests js, je pense).
fpeters force-pushed wip/66067-admin-permissions-check-all from 7ba253ff87 to 0837a36196 2023-10-04 12:22:40 +02:00 Compare
tnoel requested changes 2023-10-05 10:26:54 +02:00
tnoel left a comment
Owner
Copy Link

Mon avis c'est que j'aime pas : cocher toute la ligne ça n'arrive qu'une seule fois pour le tout premier rôle « Debug & Support » et ensuite ça ne doit jamais être coché (et surtout par Users et Roles).

Je vais faire un ticket « donner toutes les permissions au premier rôle créé » qui, à mon avis, sera encore plus satisfaisant pour les déployeurs.

(Bon après je n'ai pas bien compris le patch qui me semble plus générique que la demande, et joue sur tous les « CheckboxesTableWidget » ?)

Mon avis c'est que j'aime pas : cocher toute la ligne ça n'arrive qu'une seule fois pour le tout premier rôle « Debug & Support » et ensuite ça ne doit jamais être coché (et surtout par Users et Roles). Je vais faire un ticket « donner toutes les permissions au premier rôle créé » qui, à mon avis, sera encore plus satisfaisant pour les déployeurs. (Bon après je n'ai pas bien compris le patch qui me semble plus générique que la demande, et joue sur tous les « CheckboxesTableWidget » ?)
fpeters commented 2023-10-05 10:30:42 +02:00
Author
Owner
Copy Link

Mon avis c'est que j'aime pas : cocher toute la ligne ça n'arrive qu'une seule fois pour le tout premier rôle « Debug & Support » et ensuite ça ne doit jamais être coché (et surtout par Users et Roles).

Oui et ça fait 8 clics et ça ennuie Paul, c'est le ticket. (bien sûr il pouvait/pourrait être rejeté)

Je vais faire un ticket « donner toutes les permissions au premier rôle créé » qui, à mon avis, sera encore plus satisfaisant pour les déployeurs.

Je ne suis pas bien sûr j'ai peur de déploiement avec les rôles importés depuis un export et genre "Agent" qui se trouverait avec tout.

(Bon après je n'ai pas bien compris le patch qui me semble plus générique que la demande, et joue sur tous les « CheckboxesTableWidget » ?)

La page en question est la seule qui utilise CheckboxesTableWidget.

> Mon avis c'est que j'aime pas : cocher toute la ligne ça n'arrive qu'une seule fois pour le tout premier rôle « Debug & Support » et ensuite ça ne doit jamais être coché (et surtout par Users et Roles). Oui et ça fait 8 clics et ça ennuie Paul, c'est le ticket. (bien sûr il pouvait/pourrait être rejeté) > Je vais faire un ticket « donner toutes les permissions au premier rôle créé » qui, à mon avis, sera encore plus satisfaisant pour les déployeurs. Je ne suis pas bien sûr j'ai peur de déploiement avec les rôles importés depuis un export et genre "Agent" qui se trouverait avec tout. > (Bon après je n'ai pas bien compris le patch qui me semble plus générique que la demande, et joue sur tous les « CheckboxesTableWidget » ?) La page en question est la seule qui utilise CheckboxesTableWidget.
Some checks failed
gitea/wcs/pipeline/head There was a failure building this commit
Details
This pull request has changes conflicting with the target branch.
  • wcs/qommon/static/css/dc2/admin.scss
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
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: entrouvert/wcs#757
No description provided.
Delete Branch "wip/66067-admin-permissions-check-all"

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?