entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

toulouse-maelis: display default template values in manager (#83054) #385

Merged
nroche merged 1 commits from wip/83054-display-default-values-in-view into main 2023-11-10 10:11:56 +01:00
Conversation 1 Commits 1 Files Changed 2 +34 -46
nroche commented 2023-11-02 15:26:19 +01:00
Owner
Copy Link

Afficher les valeurs par défaut des paramètres "templates" des endpoints dans l'IHM.

Afficher les valeurs par défaut des paramètres "templates" des endpoints dans l'IHM.
nroche added 1 commit 2023-11-02 15:26:19 +01:00
nroche commented 2023-11-02 15:26:32 +01:00
Author
Owner
Copy Link
No description provided.
Screenshot 2023-11-02 at 15-21-48 Passerelle.png
57 KiB
Screenshot 2023-11-02 at 15-21-48 Passerelle.png
smihai requested changes 2023-11-02 16:12:00 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -727,3 +725,3 @@
yield item
def read_child_list_raw(self, family_id, text_template=None):
def read_child_list_raw(self, family_id, text_template):
smihai commented 2023-11-02 16:11:55 +01:00
Owner
Copy Link

Tu ne devrais pas poser la valeur {{ first_name }} {{ last_name }} par défaut ici, comme tu le fais pour les autres fonctions ?

Tu ne devrais pas poser la valeur `{{ first_name }} {{ last_name }}` par défaut ici, comme tu le fais pour les autres fonctions ?
nroche commented 2023-11-02 17:43:03 +01:00
Author
Owner
Copy Link

Non, je ne préfère pas.
J'explicite la valeur par défaut dans les prototypes des endpoints (pour que la magie de passerelle opère) seulement.
Là, c'est une simple fonction (..._raw) qui permet de centraliser les appels à maélis.
Je préfère ne pas recopier les valeurs par défaut, pour ne les maintenir qu'à un seul endroit.

Non, je ne préfère pas. J'explicite la valeur par défaut dans les prototypes des endpoints (pour que la magie de passerelle opère) seulement. Là, c'est une simple fonction (..._raw) qui permet de centraliser les appels à maélis. Je préfère ne pas recopier les valeurs par défaut, pour ne les maintenir qu'à un seul endroit.
nroche requested review from smihai 2023-11-02 17:44:32 +01:00
nroche force-pushed wip/83054-display-default-values-in-view from 322d05dd79 to fe2a9a8bc2 2023-11-02 17:52:21 +01:00 Compare
nroche changed title from toulouse-maelis: display default template values in manager (#77295) to toulouse-maelis: display default template values in manager (#83054) 2023-11-02 17:52:56 +01:00
smihai approved these changes 2023-11-02 17:55:40 +01:00
nroche added 1 commit 2023-11-10 09:53:43 +01:00
nroche force-pushed wip/83054-display-default-values-in-view from c0cbc826ce to 34ac701200 2023-11-10 09:54:52 +01:00 Compare
nroche merged commit 34ac701200 into main 2023-11-10 10:11:56 +01:00
nroche deleted branch wip/83054-display-default-values-in-view 2023-11-10 10:11:56 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
smihai
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/passerelle#385
No description provided.
Delete Branch "wip/83054-display-default-values-in-view"

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?