Produits Entr'ouvert » Combo

Valentin Deniaud a écrit :

Avec un début de patch publik-base-theme : il reste à faire un sort au lien « (supprimer) », je pense, peut-être le transformer en bouton ?

Une croix rouge ? Mais je dirai de ne pas s'en occuper ici, on ouvrira un ticket publik-base-theme une fois ceci poussé, ça déplacera la discussion au bon endroit avec les bons interlocuteurs.

Pour les patchs :

• 0001 no soucy
• 0002 :

individual_payment ne me semble pas très clair, can_pay_only_one_basket_item c'est un peu long mais on comprend tout de suite à quoi ça sert.

    url(r'^lingo/pay/(?P<item_pk>\w+)/$$', PayView.as_view(), name='lingo-pay'),

On a déjà une vue BasketItemPayView, peut-être ce serait mieux de l'utiliser ?

Actuellement de tels URLs ne sont générés que par le web-service d'ajout d'un item (et donc ça n'est jamais utilisé pour les factures, uniquement pour les items ajoutés depuis un workflow), il faut peut-être vérifier quand même que ça marchera aussi pour des RemoteItem.

        payment_url = reverse(
            'basket-item-pay-view',
            kwargs={
                'item_signature': signing_dumps(item.pk)
            })
        return JsonResponse({'result': 'success', 'id': str(item.id),
                             'payment_url': request.build_absolute_uri(payment_url)})

Ça vaut peut-être la peine d'ajouter un accesseur sur RemoteItem et BasketItem pour générer ces URLs (BasketItem.get_payment_url()).

Valentin Deniaud a écrit :

individual_payment ne me semble pas très clair, can_pay_only_one_basket_item c'est un peu long mais on comprend tout de suite à quoi ça sert.

Moi je trouve le premier assez clair et le second vraiment trop long (et je ne sais jamais de qui on parle quand les noms des attributs commencent par can_). Pour l'instant j'ai juste étendu le verbose_name, m'enfin si tu insistes je ferai le sed.

Moi aussi je comprends, je suis sur le ticket avec toi, mais demande à ton voisin de table sans lui parler du ticket ce qu'il pense que ça veut dire :)

On a déjà une vue BasketItemPayView, peut-être ce serait mieux de l'utiliser ?

Cool en plus c'est en GET, ça dégage le formulaire à actions multiples que j'avais dû bricoler.

il faut peut-être vérifier quand même que ça marchera aussi pour des RemoteItem

Je n'ai jamais joué avec ça, je n'ai pas de setup local pour tester

Il y a déjà des tests autour de ça qu'on peut copier mais..

c'est un besoin immédiat du client ?

Non tu as raison, mais alors ajoute une erreur dans le clean du formulaire pour dire que Regie.individual_payment/can_bidule_machin est incompatible avec Regie.webservice_url, qu'on garde un trace pour traiter ça plus tard.

Benjamin Dauvergne a écrit :

Moi aussi je comprends, je suis sur le ticket avec toi, mais demande à ton voisin de table sans lui parler du ticket ce qu'il pense que ça veut dire :)

Le télétravail t'a donné raison.

ajoute une erreur dans le clean du formulaire pour dire que Regie.individual_payment/can_bidule_machin est incompatible avec Regie.webservice_url

Done, branche à jour.