Projet

Général

Profil

Bug #23593

Teamnet Axel: Migrer en GenericConnectorView

Ajouté par Josué Kouka il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Josué Kouka
Version cible:
-
Début:
04 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Le connecteur fonctionne actuellement comme une app classique django. Vu que des évolutions sont attendues, il serait mieux de le migrer en GenericConnectorView

Branche avec les patchs des tickets reliés https://git.entrouvert.org/passerelle.git/log/?h=wip/teamnet_axel_generic_connector_23593


Fichiers

0001-teamnet-axel-use-GenericConnectorView-23593.patch (21 ko) 0001-teamnet-axel-use-GenericConnectorView-23593.patch Josué Kouka, 14 juin 2018 18:00
connecteur-teamnet .png (600 ko) connecteur-teamnet .png Serghei Mihai (congés, retour 15/05), 15 juin 2018 16:32

Demandes liées

Lié à Passerelle - Bug #23317: Teamnet Axel: ajouter des tests Fermé20 avril 2018

Actions
Lié à Passerelle - Bug #23607: Teamnet Axel: renvoyer une 400 quand le nameid n'est pas passéFermé04 mai 2018

Actions
Lié à Passerelle - Bug #24507: Teamnet Axel: renvoyer True quand un paiement est correctement pris en compteFermé13 juin 2018

Actions

Révisions associées

Révision e296817d (diff)
Ajouté par Josué Kouka il y a presque 6 ans

teamnet_axel: misc, fix pep8 errors (#23593)

Révision 0d894dad (diff)
Ajouté par Josué Kouka il y a presque 6 ans

teamnet axel: use GenericConnectorView (#23593)

Historique

#1

Mis à jour par Josué Kouka il y a environ 6 ans

  • Lié à Bug #23317: Teamnet Axel: ajouter des tests ajouté
#2

Mis à jour par Josué Kouka il y a environ 6 ans

  • Sujet changé de Teamnet Axel: Convertir en mode GenericConnectorView à Teamnet Axel: Migrer en GenericConnectorView
#3

Mis à jour par Josué Kouka il y a presque 6 ans

  • Lié à Bug #23607: Teamnet Axel: renvoyer une 400 quand le nameid n'est pas passé ajouté
#5

Mis à jour par Josué Kouka il y a presque 6 ans

  • Lié à Bug #24507: Teamnet Axel: renvoyer True quand un paiement est correctement pris en compte ajouté
#6

Mis à jour par Josué Kouka il y a presque 6 ans

Un patch avec l'utilisation de GenericConnectorView.

#7

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 6 ans

Il reste à retravailler le backoffice.

#8

Mis à jour par Josué Kouka il y a presque 6 ans

Serghei Mihai a écrit :

Il reste à retravailler le backoffice.

C'est fait. (branche à jour)

#9

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 6 ans

Petit couac dans la construction de l'URL de récupération des données famille:

    {% url "generic-endpoint" connector="teamnet-axel" slug=object.slug endpoint="family" rest="/" as endpoint_family %}
    <li>{% trans 'Get all family informations:' %} <a href="{{ endpoint_family }}?NameID">{{ endpoint_family }}?NameID=...</a></li>

Ça donne une URL du genre: https://dev-passerelle.entrouvert.lan/teamnet-axel/connecteur-teamnet/family//?NameID= qui n'existe pas (HTTP 404).

#10

Mis à jour par Josué Kouka il y a presque 6 ans

Serghei Mihai a écrit :

Petit couac dans la construction de l'URL de récupération des données famille:

[...]

Ça donne une URL du genre: https://dev-passerelle.entrouvert.lan/teamnet-axel/connecteur-teamnet/family//?NameID= qui n'existe pas (HTTP 404).

Je n'arrive pas à reproduire la 404. J'ai mis à jour la branche pour enlever le // par contre.

#11

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 6 ans

Tu te lies à une famille, et tu testes les endpoints un par un.

#12

Mis à jour par Josué Kouka il y a presque 6 ans

Serghei Mihai a écrit :

Tu te lies à une famille, et tu testes les endpoints un par un.

Ça donne une URL du genre: https://dev-passerelle.entrouvert.lan/teamnet-axel/connecteur-teamnet/family//?NameID= .

Cette url existe bel bien pourtant (i.e https://passerelle-fsb.test.entrouvert.org/teamnet-axel/connecteur-teamnet/).
Dans la structure c'est bien : https://<passerelle_url>/<connector>/<slug>/family/?NameID non ?

#13

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 6 ans

Je parle du double slash à la fin.

#14

Mis à jour par Josué Kouka il y a presque 6 ans

Serghei Mihai a écrit :

Je parle du double slash à la fin.

Ok, j'avais répondu

J'ai mis à jour la branche pour enlever le // par contre.

#15

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 6 ans

Ok, je vois que c'est corrigé sur la branche.
Ack.

#16

Mis à jour par Josué Kouka il y a presque 6 ans

  • Statut changé de En cours à Résolu (à déployer)
#17

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

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

Formats disponibles : Atom PDF