Support #25079
DPark: ajout de la gestion de renouvellement d'abonnement.
0%
Description
- Vérification de delai d'autorisation de renouvellement
- Vérification d'un doublon de renouvellement
- Enregistrement de la demande.
- Gestion des pièces jointes
- Récupération des données de paiement
- Notification de paiement
La branche correspondante https://git.entrouvert.org/passerelle.git/log/?h=wip/dpark_25079
Révisions associées
Historique
Mis à jour par Josué Kouka il y a presque 6 ans
- Description mis à jour (diff)
- Statut changé de En cours à Solution proposée
Tous les endpoints décrits sont implémentés dans https://git.entrouvert.org/passerelle.git/log/?h=wip/dpark_25079.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Numéro de ticket dans les messages de commit please.
Beaucoup de return {'data': True}
mais à certains endroits des trucs genre return {'data': normalize_reply(reply)}
.
Pourquoi tu renvoie pas toujours le normalize_reply(data) dans champ raw
par exemple plus de la donnée formatée dans un autre champ (pour s'éviter des tickets additionnels plus tard) ?
Tu re-inventes un peu la roue avec check_payload
(dans iws j'ai introduit json-schema qui fait plus et mieux) mais bon ...
Mis à jour par Josué Kouka il y a presque 6 ans
Emmanuel Cazenave a écrit :
Numéro de ticket dans les messages de commit please.
yep, j'ai mis en plusieurs commits juste pour la relecture. Je vais squasher avant de pousser.
Beaucoup de
return {'data': True}
mais à certains endroits des trucs genrereturn {'data': normalize_reply(reply)}
.
Pourquoi tu renvoie pas toujours le normalize_reply(data) dans champraw
par exemple plus de la donnée formatée dans un autre champ (pour s'éviter des tickets additionnels plus tard) ?
Parce qu'il y'a des cas ou le retour webservice est juste un code de retour (01) signifiant un succes et dans d'autre cas des données qui doivent renvoyées.
Tu re-inventes un peu la roue avec
check_payload
(dans iws j'ai introduit json-schema qui fait plus et mieux) mais bon ...
Un peu overkill pour la simple validation que je voulais faire.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Josué Kouka il y a presque 6 ans
- Statut changé de Solution validée à Résolu (à déployer)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
dpark: add subscription renewal management (#25079)