Florian Briand florian_briand
  • Joined on 2024-06-18
florian_briand commented on pull request P4Pillon/Krys4lide#48 2024-08-08 16:03:36 +02:00
style: Format code with fmt

Pour les into_response, j'ai l'impression qu'appeler explicitement la fonction n'est pas necessaire: https://github.com/djc/askama/blob/main/askama_axum/tests/basic.rs

Ca pourrait etre…

florian_briand commented on pull request P4Pillon/Krys4lide#48 2024-08-08 15:39:03 +02:00
style: Format code with fmt

Ça aurait valu le coup d'automatiser ça dans un githook client et voir si on pouvait ajuster les règles, pour qu'elles soient un chouilla + aligné avec notre formatage (entre autre le passage…

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:30:54 +02:00
Configurer le re-build automatique de l'app front lors de changements

Tout ce passage à base de listeners est un peu cryptique ; je pense que ça vaudrait le coup de le sortir dans une fonction au nom explicite pour gagner en lisibilité :) (get_tcp_listener par exemple)

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:30:54 +02:00
Configurer le re-build automatique de l'app front lors de changements

Ça vaut le coup de passer à la ligne, quand y'a de la programmation "fonctionnelle" comme ça, à base de layers qui s'empilent :

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:30:54 +02:00
Configurer le re-build automatique de l'app front lors de changements

Ça vaudrait le coup de "comprendre" pourquoi on exclue les requêtes htmx dans ce contexte, et de l'expliquer en docstring de la fonction

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:30:54 +02:00
Configurer le re-build automatique de l'app front lors de changements

Le pourquoi du comment vient de cette issue dont le fix a été cette PR

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:30:54 +02:00
Configurer le re-build automatique de l'app front lors de changements

Je pense qu'il faut également watch le dossier assets/ pour reload quand il y a des changements dans les fichiers ".js", ".css", etc.

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:30:44 +02:00
Configurer le re-build automatique de l'app front lors de changements

Le pourquoi du comment vient de cette issue dont le fix a été cette PR

En gros,…

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:23:17 +02:00
Configurer le re-build automatique de l'app front lors de changements

De mon côté, j'ai du mal à comprendre l'utilité des différents systèmes de reload. Je détaille pour essayer de bien comprendre :)

  • cargo watch : relance le cargo run en cas de…
florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 12:04:50 +02:00
Configurer le re-build automatique de l'app front lors de changements

Cargo watch ne supporte pas Windows 7 et inférieur

Est-ce qu'on considère ça comme ok ou est-ce un problème ? Le cas échéant, ça…

florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-08 11:52:05 +02:00
Configurer le re-build automatique de l'app front lors de changements

A quoi sert listenfd si axum serve deja les fichier et cargo watch permet d'observer le file system? Ca serait possible avec seulement cargo watch?

Extrait de la [doc de watch](https://github.…

florian_briand commented on issue P4Pillon/Krys4lide#38 2024-08-08 08:30:49 +02:00
Definition de structs et convertion automatique de la donnee renvoyee par la lib sesame vitale

Pour une bonne lisibilité de l'implémentation que tu vas faire, je te conseille de de séparer la partie "déplacement de l'existant vers la nouvelle lib" de la partie "implémentation des…

florian_briand commented on issue P4Pillon/Krys4lide#38 2024-08-07 21:57:31 +02:00
Definition de structs et convertion automatique de la donnee renvoyee par la lib sesame vitale

2: YAGNI : n'implémente pas des trucs totalement hypothétiques et non lié à un besoin : c'est une énorme source de pollution et donc de dette

3: bien vu, ça n'a pas de sens un truc à la…

florian_briand commented on issue P4Pillon/Krys4lide#44 2024-08-07 19:37:56 +02:00
Configurer le re-build automatique de l'app front lors de changements dans les fichiers html

@kosssi il est probable que les deux soient très liés et gérés automatiquement en boîte noire par une des librairies de la stack web, car c'est un essentiel de la devexp d'une telle…

florian_briand commented on issue P4Pillon/Krys4lide#38 2024-08-07 17:35:38 +02:00
Definition de structs et convertion automatique de la donnee renvoyee par la lib sesame vitale
  • Tu pourrais utiliser un Option<CarteProfessionelSante> plutôt qu'avoir une valeur AucuneCartePresente

  • Pourquoi tu as besoin de stocker le reader_slot dans la structure ?

  • Est-ce…

florian_briand commented on issue P4Pillon/Krys4lide#38 2024-08-07 13:55:12 +02:00
Definition de structs et convertion automatique de la donnee renvoyee par la lib sesame vitale

@julien.misiak pour un pharmacien, est ce qu'il y a une différence entre sa CPS physique et sa e-CPS, en terme de données ?

florian_briand commented on issue P4Pillon/Krys4lide#38 2024-08-07 11:43:24 +02:00
Definition de structs et convertion automatique de la donnee renvoyee par la lib sesame vitale

Je peux faire un document outline détaillant les choix et questionnements qu'on a eu après coup. Ou maintenant si vous voulez un support plus structure pour répertorier la discussion.

Je…

florian_briand commented on issue P4Pillon/Krys4lide#38 2024-08-07 10:19:57 +02:00
Definition de structs et convertion automatique de la donnee renvoyee par la lib sesame vitale
  1. J'ai un doute sur le fait que l'identifiant devrait appartenir à la carte CPS ou au pharmacien.
  2. Un identifiant est-il unique pour chaque pharmacien ou dépend-il de la carte ? 3.…
florian_briand opened issue P4Pillon/Krys4lide#46 2024-08-06 22:03:08 +02:00
[app] Gérer l'actualisation du statut current de la navbar