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

profile: add help text for user full name template form (#77571) #69

Merged
fpeters merged 1 commits from wip/77571-hobo-user-full-name-template-example-value into main 2023-09-04 12:13:30 +02:00
Conversation 5 Commits 1 Files Changed 1 +4
pmarillonnet commented 2023-08-21 14:28:26 +02:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2023-08-21 14:28:27 +02:00
pmarillonnet commented 2023-08-21 14:29:17 +02:00
Author
Owner
Copy Link
No description provided.
Screenshot 2023-08-21 at 14-28-45 Système.png
172 KiB
Screenshot 2023-08-21 at 14-28-45 Système.png
nroche commented 2023-08-21 14:55:41 +02:00
Owner
Copy Link

Rien à voir, mais c'est normal qu'il n'y ait pas de valeur par défaut pour ce template (dans la capture d'écran) ?
(je demande parce que je vois que dans la description du ticket qu'il est fait mention de la valeur par défaut)

Rien à voir, mais c'est normal qu'il n'y ait pas de valeur par défaut pour ce template (dans la capture d'écran) ? (je demande parce que je vois que dans la description du ticket qu'il est fait mention de la valeur par défaut)
pmarillonnet commented 2023-08-21 14:59:19 +02:00
Author
Owner
Copy Link

Rien à voir, mais c'est normal qu'il n'y ait pas de valeur par défaut pour ce template (dans la capture d'écran) ?
(je demande parce que je vois que dans la description du ticket qu'il est fait mention de la valeur par défaut)

Lorsque l’expression de gabarit n’est pas renseignée, c’est la méthode get_full_name du modèle d’utilisateur déclaré par la brique Django qui prend le dessus (c’est-à-dire, en générale, le prénom suivi du nom de l’usager).

> Rien à voir, mais c'est normal qu'il n'y ait pas de valeur par défaut pour ce template (dans la capture d'écran) ? > (je demande parce que je vois que dans la description du ticket qu'il est fait mention de la valeur par défaut) Lorsque l’expression de gabarit n’est pas renseignée, c’est la méthode `get_full_name` du modèle d’utilisateur déclaré par la brique Django qui prend le dessus (c’est-à-dire, en générale, le prénom suivi du nom de l’usager).
nroche commented 2023-08-21 15:33:36 +02:00
Owner
Copy Link

Lorsque l’expression de gabarit n’est pas renseignée, c’est la méthode get_full_name du modèle d’utilisateur déclaré par la brique Django qui prend le dessus (c’est-à-dire, en générale, le prénom suivi du nom de l’usager).

Peut-être indiquer dans l'aide, que l'on peut laisser ce template vide ?

ex: 'Template expressions including user profile attributes are accepted to override default django value (firstname + lastname), e.g. “{{ user.first_name }}{% if user.attributes.middle_name %} {{ user.attributes.middle_name }}{% endif %} {{ user.last_name }}”'

(à prendre ou à laisser, je ne suis pas fort en IHM, c'est juste que ça m'a perturbé de voir ce champ vide)

> Lorsque l’expression de gabarit n’est pas renseignée, c’est la méthode `get_full_name` du modèle d’utilisateur déclaré par la brique Django qui prend le dessus (c’est-à-dire, en générale, le prénom suivi du nom de l’usager). Peut-être indiquer dans l'aide, que l'on peut laisser ce template vide ? ``` ex: 'Template expressions including user profile attributes are accepted to override default django value (firstname + lastname), e.g. “{{ user.first_name }}{% if user.attributes.middle_name %} {{ user.attributes.middle_name }}{% endif %} {{ user.last_name }}”' ``` (à prendre ou à laisser, je ne suis pas fort en IHM, c'est juste que ça m'a perturbé de voir ce champ vide)
pmarillonnet commented 2023-08-21 15:37:54 +02:00
Author
Owner
Copy Link

Peut-être indiquer dans l'aide, que l'on peut laisser ce template vide ?

ex: 'Template expressions including user profile attributes are accepted to override default django value (firstname + lastname), e.g. “{{ user.first_name }}{% if user.attributes.middle_name %} {{ user.attributes.middle_name }}{% endif %} {{ user.last_name }}”'

(à prendre ou à laisser, je ne suis pas fort en IHM, c'est juste que ça m'a perturbé de voir ce champ vide)

En fait je n’ai rien précisé à ce sujet parce que, si vide, c’est à la discrétion de la brique. Par exemple dans authentic ce n’est pas exactement ce cas général évoqué plus haut. On gère aussi le cas où l’usager ne déclare ni prénom ni nom connus d’a2 :

    def get_full_name(self):
        """
        Returns the first_name plus the last_name, with a space in between.
        """
        full_name = '%s %s' % (self.first_name, self.last_name)
        return full_name.strip() or self.username or self.email
> Peut-être indiquer dans l'aide, que l'on peut laisser ce template vide ? > ``` > ex: 'Template expressions including user profile attributes are accepted to override default django value (firstname + lastname), e.g. “{{ user.first_name }}{% if user.attributes.middle_name %} {{ user.attributes.middle_name }}{% endif %} {{ user.last_name }}”' > ``` > (à prendre ou à laisser, je ne suis pas fort en IHM, c'est juste que ça m'a perturbé de voir ce champ vide) En fait je n’ai rien précisé à ce sujet parce que, si vide, c’est à la discrétion de la brique. Par exemple dans authentic ce n’est pas exactement ce cas général évoqué plus haut. On gère aussi le cas où l’usager ne déclare ni prénom ni nom connus d’a2 : ```python def get_full_name(self): """ Returns the first_name plus the last_name, with a space in between. """ full_name = '%s %s' % (self.first_name, self.last_name) return full_name.strip() or self.username or self.email ```
vdeniaud requested changes 2023-08-22 10:57:01 +02:00
hobo/profile/forms.py Outdated
@ -24,2 +24,4 @@
user_full_name_template = forms.CharField(
label=_('User full name template (Django)'),
help_text=_(
'''
vdeniaud commented 2023-08-22 10:56:42 +02:00
Owner
Copy Link

Ça donne un saut de ligne dans les trads, il faut splitter comme c'est fait ailleurs (par exemple hobo/sms/forms.py).

Ça donne un saut de ligne dans les trads, il faut splitter comme c'est fait ailleurs (par exemple hobo/sms/forms.py).
pmarillonnet commented 2023-08-22 11:11:17 +02:00
Author
Owner
Copy Link

Yes, c’est modifié, merci.

Yes, c’est modifié, merci.
pmarillonnet force-pushed wip/77571-hobo-user-full-name-template-example-value from e8c5c4294d to 6b7abdfb20 2023-08-22 11:11:10 +02:00 Compare
pmarillonnet requested review from vdeniaud 2023-08-22 11:11:25 +02:00
vdeniaud approved these changes 2023-08-22 11:27:29 +02:00
fpeters merged commit a1bcf1c53b into main 2023-09-04 12:13:30 +02:00
fpeters deleted branch wip/77571-hobo-user-full-name-template-example-value 2023-09-04 12:13:30 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
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/hobo#69
No description provided.
Delete Branch "wip/77571-hobo-user-full-name-template-example-value"

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?