entrouvert
/
hobo
16
0
Fork 0
Code Issues Pull Requests 18 Releases Activity

profile: add user full name template definition popup (#74507) #19

Merged
pmarillonnet merged 2 commits from wip/74507-profile-user-full-name-template-field into main 2023-02-28 15:00:06 +01:00
Conversation 2 Commits 2 Files Changed 9 +107 -5
pmarillonnet commented 2023-02-17 11:24:14 +01:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2023-02-17 11:24:14 +01:00
gitea/hobo/pipeline/pr-main Something is wrong with the build of this commit Details
gitea/hobo/pipeline/head Something is wrong with the build of this commit Details
bfe95ee76b profile: add user full name template definition popup (#74507)
aberriot approved these changes 2023-02-20 09:19:59 +01:00
aberriot left a comment
Owner
Copy Link

Testé en local, ça fonctionne bien, par contre, pas d'option pour retirer le template. Si je mets un template vide, il n'y a plus rien qui est affiché comme nom d'utilisateur. Peut-être rajouter la gestion du cas particulier ou la variable est vide pour la supprimer ? (et le test associé)

Testé en local, ça fonctionne bien, par contre, pas d'option pour retirer le template. Si je mets un template vide, il n'y a plus rien qui est affiché comme nom d'utilisateur. Peut-être rajouter la gestion du cas particulier ou la variable est vide pour la supprimer ? (et le test associé)
aberriot requested changes 2023-02-20 09:27:28 +01:00
aberriot left a comment
Owner
Copy Link

(Trompée de bouton, j'ai besoin que le point évoqué dans ma revue précédente soit traité avant validation ;)

(Trompée de bouton, j'ai besoin que le point évoqué dans ma revue précédente soit traité avant validation ;)
pmarillonnet commented 2023-02-23 10:51:44 +01:00
Author
Owner
Copy Link

Testé en local, ça fonctionne bien, par contre, pas d'option pour retirer le template. Si je mets un template vide, il n'y a plus rien qui est affiché comme nom d'utilisateur. Peut-être rajouter la gestion du cas particulier ou la variable est vide pour la supprimer ? (et le test associé)

Ouch, j’ai oublié ça, c’est embarrassant. Merci pour ta vigilance, je corrige ça et j’ajoute le test.

> Testé en local, ça fonctionne bien, par contre, pas d'option pour retirer le template. Si je mets un template vide, il n'y a plus rien qui est affiché comme nom d'utilisateur. Peut-être rajouter la gestion du cas particulier ou la variable est vide pour la supprimer ? (et le test associé) Ouch, j’ai oublié ça, c’est embarrassant. Merci pour ta vigilance, je corrige ça et j’ajoute le test.
pmarillonnet force-pushed wip/74507-profile-user-full-name-template-field from bfe95ee76b to 6972bac29e 2023-02-23 14:53:21 +01:00 Compare
pmarillonnet force-pushed wip/74507-profile-user-full-name-template-field from 6972bac29e to 5fc078550b 2023-02-23 14:58:51 +01:00 Compare
pmarillonnet commented 2023-02-23 15:10:02 +01:00
Author
Owner
Copy Link

Ouch, j’ai oublié ça, c’est embarrassant. Merci pour ta vigilance, je corrige ça et j’ajoute le test.

Voilà, c’est corrigé dans la branche. Plutôt que de faire un cas particulier pour cet héritage du VariablesFormMixin d’hobo, simplement gérer dans l’appli user_name le cas où la variable de gabarit est présente mais vide.

> Ouch, j’ai oublié ça, c’est embarrassant. Merci pour ta vigilance, je corrige ça et j’ajoute le test. Voilà, c’est corrigé dans la branche. Plutôt que de faire un cas particulier pour cet héritage du `VariablesFormMixin` d’hobo, simplement gérer dans l’appli `user_name` le cas où la variable de gabarit est présente mais vide.
pmarillonnet requested review from aberriot 2023-02-23 15:10:05 +01:00
aberriot approved these changes 2023-02-27 09:42:55 +01:00
aberriot left a comment
Owner
Copy Link

Retesté en local, c'est tout bon !

Retesté en local, c'est tout bon !
pmarillonnet force-pushed wip/74507-profile-user-full-name-template-field from 5fc078550b to 111591a0b0 2023-02-28 14:53:14 +01:00 Compare
pmarillonnet merged commit 111591a0b0 into main 2023-02-28 15:00:06 +01:00
pmarillonnet deleted branch wip/74507-profile-user-full-name-template-field 2023-02-28 15:00:07 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
aberriot
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/hobo#19
No description provided.
Delete Branch "wip/74507-profile-user-full-name-template-field"

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?