Florian Briand florian_briand
  • Joined on 2024-06-18
florian_briand created branch feat/handle_errors_rustly in P4pillon/Krys4lide 2024-08-13 23:01:00 +02:00
florian_briand opened issue P4pillon/Krys4lide#50 2024-08-10 12:33:16 +02:00
Correction des étiquettes gitea pour utiliser les nouveaux noms des composants
florian_briand pushed to html_auto_reload at P4pillon/Krys4lide 2024-08-09 13:00:11 +02:00
dfdf0d3bae fixup! refacto: extract livereload layer setup into a function
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.…