-
Notifications
You must be signed in to change notification settings - Fork 91
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ETQ usager je vois les infos de contact de mon groupe instructeur avant dépôt du dossier, une fois le(s) champ(s) de routage rempli(s) #10430
Conversation
95ebd69
to
9a0fb7b
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10430 +/- ##
=======================================
Coverage 80.23% 80.23%
=======================================
Files 1233 1233
Lines 26263 26265 +2
Branches 4716 4718 +2
=======================================
+ Hits 21071 21075 +4
+ Misses 5192 5190 -2 ☔ View full report in Codecov by Sentry. |
1e53d06
to
6794054
Compare
846992f
to
3800ed5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
un petit test system pour valider qu'on cassera pas ça ds le futur ? sinon c'est top
Ça marche 👍 j'ai complété un system spec existant |
d7f5e7d
to
6f43f66
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
est-ce qu'on a un moyen (facile) de savoir si le routage / les infos de contact a changé ? Pour éviter de render ce contact information dans 99% des cas où ce ne sera pas nécessaire, que ce soit côté serveur ou côté navigateur et conserver une réactivité maximale de l'autosave (sachant que ce serait déjà largement perfectible)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@colinux dans le dernier commit j‘ai ajouté une condition pour faire l‘update du footer seulement si on a touché à un champ de routage, mais ça fait péter un test de system : spec/system/users/brouillon_spec.rb:9
. Sachant que ça a l‘air d‘être un flaky test (il échoue sur main en local sur mon poste). J‘ai testé différents fixs, sans succès, je mets ça en pause pour l‘instant. Si jamais tu as une idée...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@colinux Je me demande si c'est une bonne idée d'introduire ce genre d'optimisations ? J'ai l'impression que nous avons beaucoup de conditions complexes dans les turbo_stream. Je veux bien qu'on en parle au point tech. J'aimerais bien qu'on utilise quand c'est possible le nouveau turbo_stream
refresh
avec morph dans la plupart des cas (l'autosave c'est probablement pas le bon candidat, mais même ici je pense qu'on peut simplifier)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l'optimisation n'étant pas tant pour le temps serveur, mais pour éviter que les navigateurs refassent des manips/comparaisons de DOM inutiles (je pense en particulier aux mobiles low-end chez qui la page de la démarche lag déjà). Cela dit il y a peut-être des astuces turbo/morph que j'ignore qui font que c'est déjà pas cher du tout, ok pour en causer
0d3b55c
to
dd2a0f0
Compare
f41cb46
to
d8271cc
Compare
15965b4
to
12c0324
Compare
ad6c67d
to
550ba1e
Compare
e9ba17c
to
a342716
Compare
…es has been updated
0bc2e36
to
a04749c
Compare
closes #10028
Démo du routage et mise à jour du foter en autosave :
Capture.video.du.21-05-2024.09.52.26.webm