Masquer le bouton Télécharger si Bigup est bien actif #4483

Si Bigup est bien lancé, c'est que le JS marche bien etc. On a donc la zone de drop, et même si on veut pas drop, ya le bouton pour ouvrir la boite de fichiers. Ensuite que l'on drop ou que l'on fasse pas Parcourir, dans les deux cas ça lance l'upload immédiatement. On ne devrait donc jamais voire le bouton Télécharger, puisque ça incite à cliquer dessus, ce qui fait tout planter non ? (si on clique pendant que ça progresse en JS)

Vu que là on s'apprête à l'ajouter dans la distrib officielle, ça serait bien de le corriger avant car c'est un gros truc d'UI qui se voit et qui vaut pour tout le monde.

Et le patch est ici : https://github.com/marcimat/bigup/pull/13

J'ai fait un fork et une branche ici https://git.spip.net/RealET/bigup/src/branch/issue%234483

J'ai pu fusionner cette branche avec mon fork (https://git.spip.net/RealET/bigup)
Mais je n'ai pas accès à la demande de fusion de cette branche avec https://git.spip.net/spip/bigup

Et maintenant que https://git.spip.net/spip/bigup est l'original et non plus le miroir de github, je ne peut toujours pas faire la PR dessus.

parce que quoi ? ya pas le bouton ?

Y'a le bouton, je peux dire que je choisi spip:master dans la liste déroulante.
Et là, je tombe sur une 404 : https://git.spip.net/spip/bigup/compare/master...RealET/bigup:issue%234483

mais t'as vraiment nommé ta branche avec un # ? Le nom de la branche est utilisée dans l'URL ensuite, et donc forcément # ça veut dire une ancre dans une URL… donc du coup l'URL générée est en fait "bigup/issue" à la fin, ce qui n'existe pas, non ?

Ben oui, personne ne m'a dit que c'était impossible, alors je l'ai fait ;-)

Bé faut que tu la renommes donc

J'ai renommé
Je vais dans https://git.spip.net/RealET/bigup/compare/master...issue4483
Je choisi de faire une nouvelle demande de fusion.
J'indique comme destination : spip:Master (j'ai pas d'autres choix pour SPIP).
Et je tombe sur une 404
https://git.spip.net/spip/bigup/compare/master...RealET/bigup:issue4483 : La page que vous essayez d'atteindre n'existe pas ou vous n'êtes pas autorisé à la voir.

`RastaPopoulos : est-ce que tu peux essayer toi, qui doit être "Autorisé à la voir" ?

Bonjour

La fonctionnalité PR n'avait pas été activée suite à l'indépendance du projet sur la forge.
Cela est fait.

La demande de fusion devrait être disponible maintenant.

La PR est donc là, à valider #1
Statut changé à En cours

Intégré
Statut changé à Fermé