Related to #133
Tout fonctionne très bien, j'ai juste noté quelques chemins manquants :
Related to #150
J'approuve d'ors et déjà, ma seule proposition étant optionnelle.
Related to #138
Cette ligne gagnerait peut-être à être hors du else
pour recaler @pivot_date
dans tous les cas à la date de début du rendez-vous qui va être créé ?
Juste une toute petite proposition d'amélioration au milieu de ce gros boulot, et une petite incohérence de wording au hasard des tests dans l'IHM.
Juste une petite incohérence de wording repérée en passant : un certificat envoyé apparaît au survol comme Certificat délivré
(fr.yml:42
).
Ici, je construirais la lien avec en params
le contenu de session[:appointments_filter]
pour que l'URL dans la barre d'adresse …/appointments/archives
reflète les filtres qui seront effectivement appliqués (et avec ça, plus besoin de double ||
dans le contrôleur).
Related to #132
Je n'avais pas réalisé que c'était des ActionController::Parameters
permit
suffit :
@item_params = params.permit('consultation_type_id', 'intervention_domain_id')
Testé et validé. Il me manquait la date du jour mais elle est dans le fil d'Ariane, qui n'apparaît pas encore ici.
Juste une réponse sur l'incohérence ".id
ou pas" et je peux approuver.
Juste un détail : item_params
n'est pas très parlant pour moi. Considérant son utilisation, j'imaginerais bien un navigation_params
ou un context_params
. Mais je pourrai vivre avec
Il y a une incohérence dans les valeurs. Tu prends .id
pour le type de consultation mais pas pour le domaine d'intervention. Si ce sont bien les ID qui t'intéressent, peut-être peux-tu simplifier cette ligne avec un Hash#slice ?
@item_params = params.slice('consultation_type_id', 'intervention_domain_id')
Effectivement, d'autant plus que ce sera peut-être même plus visible directement sur le jour de calendrier plutôt que dans un coin de l'écran.
Related to #149
C'est l'info qu'il me manquait
Le ticket, oui, mais pas sa résolution. Il y a de la magie derrière turboTemporary
?
J'avoue que je ne comprends pas du tout cette PR… Peux-tu m’expliquer de quoi il retourne ?
Cette modification fonctionne tant qu'on n'a pas dédoublonné les OPJ comme nous l'avons fait pour les patients