Skip to content
Extraits de code Groupes Projets

WIP formidable_ts_issue_36_crayon_fichier

Fermé Maïeul a demandé de fusionner gh-fdf4c590/183/unknown/refs/pull/183/head vers master

hop @patrice4

cette PR permet de crayonner les champs fichier (sauf suppression de fichier, auquel cas modifier pour l'instant directement toute la réponse).

cf formidable_tablesorter#36 (closed)

Rapports de requête de fusion

Fermée par avatar (janv. 10, 2025 4:01am UTC)

Loading

Activité

Filtrer l'activité
  • Approbations
  • Assignés et relecteurs
  • Commentaires (des bots)
  • Commentaires (des utilisateurs)
  • Branches et validations
  • Modifications
  • Labels
  • État de verrouillage
  • Mentions
  • État de la demande de fusion
  • Suivi
  • @maieul a ajouté 5 révisions : 2fe7b4c2 9b4fe515556212dc485d2fca502fd8e9c0ba0382 911f7259b436a574046f6a86574b9fe0a186c398 acb8b8c4a7ea41ba60692bc800276483b972c955 18f5c17fcd0203cd67a9d379f6b901620c97f0d5

  • Hello, ça semble marcher, plus d'erreur et le crayonnage permet bien d'ajouter des fichiers. Par contre un second clic sur le crayon et là, plus de fichier. Si je valide pour voir, alors comme attendu ensuite, tous les fichiers ont disparu.

  • Auteur Maintainer

    oui, c'est lié au fait que les crayons ne sont pas des vrais formulaires CVT (scrgn,scgnr), donc c'est compliqué de pouvoir précharger ce qui existe deja en base facilement. Je vais voir si j'arrive à améliorer cela, mais c'est clairement pas ma priorité.

  • @maieul a force-pushed 2 révisions : 18f5c17fcd0203cd67a9d379f6b901620c97f0d5 1da41ac613747d76eb75689823cf1d01201b200d

  • @maieul a remplacé le titre formidable_ts_issue_36_crayon_fichier par WIP formidable_ts_issue_36_crayon_fichier

  • Auteur Maintainer

    bon, j'ai pu avancer un peu: le controleur est bon, reste à affiner le traitement.

  • @maieul a force-pushed 2 révisions : 1da41ac613747d76eb75689823cf1d01201b200d 93dc5ad3

  • Auteur Maintainer

    La marche bien je pense.

    La seule chose que je ne fais pas par rapport à simplement éditer la réponse, c'est de demander confirmation lorsqu'on supprime un fichier.

    Ce serait sans doute trop complexe à implementer vu le fonctionnement de crayon.

    Pour moi c'est testable, et le cas échéant mergeable.

    Bon un jour faudra revoir tout le code de cvtupload, mais c'est un autre problème.

  • Auteur Maintainer

    et donc @patrice4 a tester

  • @patrice4 a approuvé cette PR (93dc5ad3)

    QQ tests positifs, merci !

  • @maieul a fermé cette PR

  • Auteur Maintainer

    C'est intégré dans master, la prochane sortie d'une version de formidable est attendue pour la fin du mois.

Veuillez vous inscrire ou vous connecter pour répondre
Chargement en cours