Development #33198
planitec : exposer la recherche de créneaux sur des jours spécifiques de la semaine
0%
Description
L'API de planitec offre cette possibilité, le connecteur cache cette fonctionnalité et offre simplement une recherche jours de semaine/week-end. Garder ça mais accepter aussi une liste de jours arbitraire.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Fichier 0001-planitec-expose-days-restriction-parameter-as-is-331.patch 0001-planitec-expose-days-restriction-parameter-as-is-331.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Petit changement d'API au passage, j'expose celle de planitec telle quelle :
- pas de weekdays : aucune restriction sur les jours
- weekdays : on spécifie les jours qui nous intéressent
Mis à jour par Thomas Noël il y a presque 5 ans
Ultra détail, mais ça m'est arrivé ailleurs, au lieu du simple :
for day in weekdays.split(','):
tu pourrais avoir un truc plus "intelligent" qui gère des vides entre virgules (le coup "classique" d'un appel avec {{form_var_x}},{{form_var_y}} et paf, form_var_y n'existe pas) :
for day in [d.strip() for d in weekdays.split(',') if d.strip()]:
Aussi, au niveau de la description 'description': _('Week days'), je compléterai : 'description': _('Week days, comma separated list of integers beetween 0 (sunday) and 6'), (si 0 est bien le dimanche, ce dont j'ai l'impression)
Bon, juste des détails quoi... Je valide quand même, tu peux pousser ainsi parce que ça marchera dans les cas d'usage prévus (qui sont limités).
Mis à jour par Thomas Noël il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Poussé avec les modifs demandées.
commit 9396e907da65fc4b83892f24f0a2fdfef8ed5072
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Mon May 20 16:57:34 2019 +0200
planitec: expose days restriction parameter as is (#33198)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
planitec: expose days restriction parameter as is (#33198)