[patch] (v2.0.0) Wrong display of TabbedForm component

[mguihal]

Hello,
Durant mes phases de tests et recettes de la v2, je suis tombé sur un bug embêtant lors de l'édition d'une ressource avec un formulaire tabulaire. En fait en cliquant sur "Editer", le formulaire d'édition ne s'affiche pas la première fois, il faut ensuite cliquer sur un des onglets pour qu'il s'affiche.

Je vous joins une vidéo pour visualiser.

Enregistrement.de.l.ecran.2024-04-28.a.22.30.15.mov

Ce qu'on peut voir c'est que l'url change entre le clic sur le bouton "Editer", et le clic sur un des onglets, sur le décodage de l'url. Celle-ci passe de http://localhost:5173/Organization/http%3A%2F%2Flocalhost%3A3000%2Forganizations%2Ftest à http://localhost:5173/Organization/http:%2F%2Flocalhost:3000%2Forganizations%2Ftest (noter le non-échappement du caractère :. Malgré plusieurs heures passées dessus, je n'ai pas réussi à comprendre exactement la source du problème. Ca semble lié à la façon dont la lib react-router-dom décode les urls, mais je n'ai pas réussi à trouver de changement précis entre les versions sur master et sur la branche v2...

EDIT : J'ai trouvé le problème initial ici : remix-run/react-router#11052. React-router a fait un breaking-change dans une version mineure. C'est sensé être revert sur les dernières versions mais visiblement il y a encore quelques soucis...

Dans tous les cas, ce bug renforce le fait qu'il va falloir s'atteler à la réécriture des urls pour ne plus avoir des url complètes dedans, comme mentionnés auparavant ici : #146, assemblee-virtuelle/semapps#636 ou assemblee-virtuelle/semapps#1222

Ma proposition de résolution est de désactiver le routing des formulaires à onglets. Le changement est que l'url ne changera pas quand on changera d'onglet, et qu'il n'y aura plus de /2 ou /3 ajoutés dans l'url. Pour moi c'est presque bénéfique car le /2 ou /3 n'apportait pas grand chose, une future évolution sera plutôt, pour les formulaire qui auront toujours besoin d'onglets, de créer des routes plus compréhensives, du style /members par exemple.

[simonLouvet]

Si je comprends bien @mguihal , cette PR implémente la non-gestion de la navigation par onglet depuis l'url.
Est-ce que ça résout également le problème du premier chargement non fonctionnel de la page par navigation

[srosset81]

J'ai eu un problème similaire avec Mastopod, sauf que le bug était au niveau de ViteJS et le mettre à jour avait suffit pour régler le problème (si je me rappelle bien). Archipelago utilise une version encore plus récente de ViteJS donc ça ne doit pas venir de là.

Si syncWithLocation: false résoud le problème temporairement, cela me semble OK, mais ce serait quand même bien de le réactiver après car c'est pratique quand on veut mettre un lien directement vers un certain onglet.

L'usage des slugs au lieu des URIs n'est pas une solution complète, car on aura toujours des cas où on voudra lire des resources distantes et où il faudra donc passer l'URI complète. On ne pourra utiliser des slugs que pour les données d'un seul serveur.