Projet

Général

Profil

Development #6651

Ajouter le support du porte-document fargo sur le modèle du support pour le porte-doc MSP

Ajouté par Benjamin Dauvergne il y a environ 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
09 mars 2015
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

Révisions associées

Révision b3bf4b7c (diff)
Ajouté par Benjamin Dauvergne il y a environ 9 ans

add fargo dropbox support to file fields (#6651)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Fichier 0001-allow-form-recipients-to-pick-files-from-their-fargo.patch ajouté
  • % réalisé changé de 0 à 50

Version préliminaire que je dois encore tester mais que je mets quand même au cas où je finirai sous un bus d'ici là.

#2

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Fichier 0001-allow-form-recipients-to-pick-files-from-their-fargo.patch supprimé
#3

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Fichier 0001-allow-form-recipients-to-pick-files-from-their-fargo.patch ajouté
  • % réalisé changé de 50 à 100
  • Patch proposed changé de Non à Oui

Voilà patch testé et intégré à un auquo master et testé sur la release 0.2 de fargo, à relire.

#4

Mis à jour par Frédéric Péters il y a environ 9 ans

Détails.

        return get_publisher().get_site_option('fargo')

Je préférerais fargo_url comme nom de variable.

        r += htmltext('<pre>Token: %s</pre>') % token

J'imagine que c'est du debug.

        r += htmltext('<script>window.top.document.fargo_set_token("%s", "%s");</script>' % (
            token, title))

On peut imaginer que le titre d'un document contienne des guillemets, et que ça foire ?

    iframe.attr({
        width: +width,
        height: +height,

Je comprends l'idiome mais ce serait quand même plus clair avec parseInt(), je trouve.

#5

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Fichier 0001-allow-form-recipients-to-pick-files-from-their-fargo.patch supprimé
#6

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

J'ai intégré toutes les remarques. Pour les chaînes à interpoler dans le JS inline, j'ai utilisé json.dumps.

#7

Mis à jour par Frédéric Péters il y a environ 9 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit b3bf4b7c32e9c3fba531ac12628388124a9717e9
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Mar 9 12:42:05 2015 +0100

    add fargo dropbox support to file fields (#6651)
#8

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF