entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 35 Releases Activity

manage: split apiclient config edition in separate tabs (#81334) #131

Open
pmarillonnet wants to merge 2 commits from wip/81334-manage-apiclient-edit-tabs into wip/81845-bo-form-error-on-hidden-tab
pull from: wip/81334-manage-apiclient-edit-tabs
merge into: entrouvert:wip/81845-bo-form-error-on-hidden-tab
Conversation 8 Commits 2 Files Changed 7 +181 -59
pmarillonnet commented 2023-09-20 10:37:05 +02:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2023-09-20 10:37:05 +02:00
gitea/authentic/pipeline/head There was a failure building this commit Details
e3673726f4 [wip] manage: split apiclient config edition in separate tabs (#81334)
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from e3673726f4 to 9a49e23494 2023-09-20 11:51:42 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 9a49e23494 to bf57daab1a 2023-09-21 09:57:35 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from bf57daab1a to 524bf2cd59 2023-09-21 11:46:05 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 524bf2cd59 to 411a932972 2023-09-21 12:01:27 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 411a932972 to 7fca3e2f9b 2023-09-25 14:13:35 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 7fca3e2f9b to 728fce4b6f 2023-09-26 10:55:19 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 728fce4b6f to a9d83b6696 2023-09-26 11:59:58 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from a9d83b6696 to 00ea85816e 2023-09-26 12:11:26 +02:00 Compare
pmarillonnet changed title from WIP: manage: split apiclient config edition in separate tabs (#81334) to manage: split apiclient config edition in separate tabs (#81334) 2023-09-26 12:16:56 +02:00
pmarillonnet commented 2023-09-26 12:19:54 +02:00
Author
Owner
Copy Link

Voilà l’approche qui me paraît être la bonne :

· On décline le formulaire en onglets avec un onglet de paramétrage avancé qui contient la restriction aux données anonymisées et celle à certains attributs de profil.

· Cet affichage par onglet n’existe que pour l’édition de clients existants. Pour la création d’un nouveau client, on ne s’embarrasse pas de ces cas particuliers et on n’affiche qu’un formulaire réduit (correspondant à ce qui est affiché dans le premier onglet du formulaire d’édition), c’est-à-dire sans les options avancées de restriction d’accès.

Voilà l’approche qui me paraît être la bonne : · On décline le formulaire en onglets avec un onglet de paramétrage avancé qui contient la restriction aux données anonymisées et celle à certains attributs de profil. · Cet affichage par onglet n’existe que pour l’édition de clients existants. Pour la création d’un nouveau client, on ne s’embarrasse pas de ces cas particuliers et on n’affiche qu’un formulaire réduit (correspondant à ce qui est affiché dans le premier onglet du formulaire d’édition), c’est-à-dire sans les options avancées de restriction d’accès.
pmarillonnet commented 2023-09-26 12:25:54 +02:00
Author
Owner
Copy Link
No description provided.
apiclient_onglet1.png
64 KiB
apiclient_onglet2.png
60 KiB
apiclient_onglet1.png apiclient_onglet2.png
bdauvergne commented 2023-09-26 15:10:22 +02:00
Owner
Copy Link

Ok, y en a d'autres des forms comme ça ailleurs ? J'ai un peu un doute sur l'ergonomie du truc si on a une erreur sur la première tab et qu'on est sur la deuxième.

Ok, y en a d'autres des forms comme ça ailleurs ? J'ai un peu un doute sur l'ergonomie du truc si on a une erreur sur la première tab et qu'on est sur la deuxième.
fpeters commented 2023-09-26 15:15:44 +02:00
Owner
Copy Link

On utilise communément ça dans combo et w.c.s. et sur une erreur on bascule sur l'onglet avec le champ en erreur. (je n'ai pas regardé ce qui était fait dans ce patch).

Ok c’est soit, très probable, un oubli de ma part, soit, moins probable, un oubli de façon générale dans le BO a2. Je regarde.

On utilise communément ça dans combo et w.c.s. et sur une erreur on bascule sur l'onglet avec le champ en erreur. (je n'ai pas regardé ce qui était fait dans ce patch). Ok c’est soit, très probable, un oubli de ma part, soit, moins probable, un oubli de façon générale dans le BO a2. Je regarde.
pmarillonnet commented 2023-09-26 15:16:25 +02:00
Author
Owner
Copy Link

Ok, y en a d'autres des forms comme ça ailleurs ? J'ai un peu un doute sur l'ergonomie du truc si on a une erreur sur la première tab et qu'on est sur la deuxième.

C’est très largement pompé sur ce qui est déjà fait dans le paramétrage des moyens d’authentification.
Cela dit oui, bien vu, je ne crois que pas la gestion des onglets dans le BO a2 de façon générale gère la bascule automatique sur l’onglet en erreur à la soumission du formulaire. Je vais regarder si on peut corriger ça partout.

> Ok, y en a d'autres des forms comme ça ailleurs ? J'ai un peu un doute sur l'ergonomie du truc si on a une erreur sur la première tab et qu'on est sur la deuxième. C’est très largement pompé sur ce qui est déjà fait dans le paramétrage des moyens d’authentification. Cela dit oui, bien vu, je ne crois que pas la gestion des onglets dans le BO a2 de façon générale gère la bascule automatique sur l’onglet en erreur à la soumission du formulaire. Je vais regarder si on peut corriger ça partout.
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 00ea85816e to a3a11bfe42 2023-10-02 09:19:46 +02:00 Compare
pmarillonnet commented 2023-10-02 09:32:29 +02:00
Author
Owner
Copy Link

je ne crois que pas la gestion des onglets dans le BO a2 de façon générale gère la bascule automatique sur l’onglet en erreur à la soumission du formulaire. Je vais regarder si on peut corriger ça partout.

C’est bien cela, ce n’est pas spécifique à cette PR, c’est un souci dans le modèle d’onglets du BO a2. J’ai fait #81845.

> je ne crois que pas la gestion des onglets dans le BO a2 de façon générale gère la bascule automatique sur l’onglet en erreur à la soumission du formulaire. Je vais regarder si on peut corriger ça partout. C’est bien cela, ce n’est pas spécifique à cette PR, c’est un souci dans le modèle d’onglets du BO a2. J’ai fait #81845.
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from a3a11bfe42 to 6c6d09b9c7 2023-10-02 16:46:13 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 6c6d09b9c7 to 39ffbc3b65 2023-10-02 17:01:48 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 39ffbc3b65 to b56078b855 2023-10-03 09:53:56 +02:00 Compare
pmarillonnet changed title from manage: split apiclient config edition in separate tabs (#81334) to manage: split apiclient config edition in separate tabs (#81334) 2023-10-03 10:07:41 +02:00
pmarillonnet changed target branch from main to wip/81845-bo-form-error-on-hidden-tab 2023-10-03 10:07:42 +02:00
pmarillonnet commented 2023-10-03 10:08:39 +02:00
Author
Owner
Copy Link

je ne crois que pas la gestion des onglets dans le BO a2 de façon générale gère la bascule automatique sur l’onglet en erreur à la soumission du formulaire. Je vais regarder si on peut corriger ça partout.

C’est bien cela, ce n’est pas spécifique à cette PR, c’est un souci dans le modèle d’onglets du BO a2. J’ai fait #81845.

#81845 est bon pour relecture, ça s’articule avec #81867 côté gadjo.
Ici on se base sur ces deux tickets pour proposer quelque chose spécifique au formulaire d’édition d’un client d’API.

> > je ne crois que pas la gestion des onglets dans le BO a2 de façon générale gère la bascule automatique sur l’onglet en erreur à la soumission du formulaire. Je vais regarder si on peut corriger ça partout. > > C’est bien cela, ce n’est pas spécifique à cette PR, c’est un souci dans le modèle d’onglets du BO a2. J’ai fait #81845. #81845 est bon pour relecture, ça s’articule avec #81867 côté gadjo. Ici on se base sur ces deux tickets pour proposer quelque chose spécifique au formulaire d’édition d’un client d’API.
pmarillonnet changed title from manage: split apiclient config edition in separate tabs (#81334) to WIP: manage: split apiclient config edition in separate tabs (#81334) 2023-10-19 10:54:30 +02:00
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from b56078b855 to 83698b7455 2023-10-19 10:54:55 +02:00 Compare
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 5fe6a350ce to 2362a409af 2023-10-19 11:25:06 +02:00 Compare
pmarillonnet changed title from WIP: manage: split apiclient config edition in separate tabs (#81334) to manage: split apiclient config edition in separate tabs (#81334) 2023-10-19 11:33:18 +02:00
pmarillonnet force-pushed wip/81334-manage-apiclient-edit-tabs from 2362a409af to 10269d0d0d 2023-10-30 11:52:52 +01:00 Compare
bdauvergne commented 2023-12-20 12:20:52 +01:00
Owner
Copy Link

Je valide mais je n'ai pas trouvé d'équivalent dans les autres briques; dans combo les onglets ont des formulaires indépendants, dans w.c.s., c'est w.c.s., donc on ne peut pas s'inspirer ou comparer non plus.

Je valide mais je n'ai pas trouvé d'équivalent dans les autres briques; dans combo les onglets ont des formulaires indépendants, dans w.c.s., c'est w.c.s., donc on ne peut pas s'inspirer ou comparer non plus.
bdauvergne approved these changes 2023-12-20 12:21:01 +01:00
All checks were successful
gitea/authentic/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/81334-manage-apiclient-edit-tabs wip/81845-bo-form-error-on-hidden-tab
git pull origin wip/81334-manage-apiclient-edit-tabs

Step 2:

Merge the changes and update on Gitea.
git checkout wip/81845-bo-form-error-on-hidden-tab
git merge --no-ff wip/81334-manage-apiclient-edit-tabs
git push origin wip/81845-bo-form-error-on-hidden-tab
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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/authentic#131
No description provided.
Delete Branch "wip/81334-manage-apiclient-edit-tabs"

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?