Florian Briand florian_briand
  • Joined on 2024-06-18
florian_briand pushed to html_auto_reload at P4Pillon/Krys4lide 2024-08-09 12:55:32 +02:00
60a409f20a refacto: extract livereload layer setup into a function
dcb4a7680e refacto: extract TCP Listener building into a dedicated function
Compare 2 commits »
florian_briand commented on pull request P4Pillon/Krys4lide#47 2024-08-09 00:46:58 +02:00
Configurer le re-build automatique de l'app front lors de changements

Je ne suis pas fan de ce genre de documentation "séparée" qui évoque des détails d'implémentation : on est sûr que dès qu'on va toucher à l'implementation, on oubliera de mettre à jour ce readme, et paf, ça devient le bordel.

florian_briand commented on issue P4Pillon/Krys4lide#46 2024-08-08 17:53:54 +02:00
[app] Gérer l'actualisation du statut current de la navbar

Ressource HTMX qui propose peut etre une solution: https://htmx.org/examples/tabs-hateoas/

Ca correspond ?

Trop bien, bonne idée les exemples HTMX! Je vais creuser cet exemple, ils…

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 ?