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

wip/87489-atal-localization #481

Merged
ecazenave merged 1 commits from wip/87489-atal-localization into main 2024-03-01 10:54:22 +01:00
Conversation 0 Commits 1 Files Changed 2 +5 -3
ecazenave commented 2024-02-26 18:08:10 +01:00
Owner
Copy Link
No description provided.
ecazenave added 2 commits 2024-02-26 18:08:11 +01:00
gitea/passerelle/pipeline/head This commit looks good Details
7b67ab9ab1 atal_rest: accept string and float for geoloc (#87484)
gitea/passerelle/pipeline/head This commit looks good Details
37ea0e0abc atal_rest: change a parameter name (#87489)
ecazenave changed title from WIP: wip/87489-atal-localization to wip/87489-atal-localization 2024-02-26 19:30:41 +01:00
tnoel requested changes 2024-02-29 14:48:25 +01:00
Dismissed
passerelle/apps/atal_rest/models.py Outdated
@ -308,0 +320,4 @@
for param, atal_param in float_params.items():
param_value = post_data.get(param, '')
if param_value:
try:
tnoel commented 2024-02-29 14:45:23 +01:00
Owner
Copy Link

Ici pour se faciliter la tâche, si param_value est de type str tu pourrais y faire un replace(',', '.') parce qu'il n'est pas toujours évident que les gabarits Django de w.c.s. n'envoie pas des flottants "francisés", avec des virgules. Ca sera toujours un ticket de support en moins :)

Ici pour se faciliter la tâche, si param_value est de type str tu pourrais y faire un replace(',', '.') parce qu'il n'est pas toujours évident que les gabarits Django de w.c.s. n'envoie pas des flottants "francisés", avec des virgules. Ca sera toujours un ticket de support en moins :)
passerelle/apps/atal_rest/models.py Outdated
@ -350,3 +370,3 @@
'description': 'Description',
'keywords': 'Keywords',
'localisation': 'Localisation',
'localisation': 'Localization',
tnoel commented 2024-02-29 14:48:08 +01:00
Owner
Copy Link

En voyant ça, tu es sûr de ne pas pouvoir mettre "localization" dans la clé attendue en entrée ? Ca permettrait de respecter la logique des autres paramètres.

Au pire, en ajoutant un de quoi gérer aussi "localisation" quand il est reçu, mais que le schéma ne fasse bien mention que de "localization".

En voyant ça, tu es sûr de ne pas pouvoir mettre "localization" dans la clé attendue en entrée ? Ca permettrait de respecter la logique des autres paramètres. Au pire, en ajoutant un de quoi gérer aussi "localisation" quand il est reçu, mais que le schéma ne fasse bien mention que de "localization".
ecazenave commented 2024-02-29 15:44:05 +01:00
Author
Owner
Copy Link

Remarque prise ne compte, build en erreur mais c'est https://dev.entrouvert.org/issues/87625

Remarque prise ne compte, build en erreur mais c'est https://dev.entrouvert.org/issues/87625
ecazenave force-pushed wip/87489-atal-localization from 37ea0e0abc to 7689bad770 2024-02-29 15:23:10 +01:00 Compare
tnoel approved these changes 2024-03-01 10:11:06 +01:00
ecazenave force-pushed wip/87489-atal-localization from 7689bad770 to 1570ba1f3f 2024-03-01 10:37:51 +01:00 Compare
ecazenave merged commit 1570ba1f3f into main 2024-03-01 10:54:22 +01:00
ecazenave deleted branch wip/87489-atal-localization 2024-03-01 10:54:23 +01:00
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/passerelle#481
No description provided.
Delete Branch "wip/87489-atal-localization"

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?