entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

connecteur avis-imposition (#43479) #199

Closed
bdauvergne wants to merge 1 commits from wip/43479-connecteur-avis-imposition into main
pull from: wip/43479-connecteur-avis-imposition
merge into: entrouvert:main
Conversation 1 Commits 1 Files Changed 9 +972
bdauvergne commented 2023-04-13 12:56:45 +02:00
Owner
Copy Link

Return value is the same as API particulier for the avis-imposition
endpoint, except for some data massaging put into alternative keys:

  • dates are converted to ISO format and put in keys with @_iso@ suffix,
  • family situation is simplified to ASCII and put in key with @_simple@
    suffix,
  • addresse is returned as oneline joined with newlines as in API
    particulier, but also with individual lines separated into 3 keys :
    adresse1, adresse2, adresse3.
Return value is the same as API particulier for the avis-imposition endpoint, except for some data massaging put into alternative keys: * dates are converted to ISO format and put in keys with @_iso@ suffix, * family situation is simplified to ASCII and put in key with @_simple@ suffix, * addresse is returned as oneline joined with newlines as in API particulier, but also with individual lines separated into 3 keys : adresse1, adresse2, adresse3.
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from b82ec3cd3d to dc6b09d98b 2023-04-13 13:35:42 +02:00 Compare
bdauvergne changed title from WIP: connecteur avis-imposition (#43479) to connecteur avis-imposition (#43479) 2023-04-13 13:35:53 +02:00
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from dc6b09d98b to bea92d1ee5 2023-04-13 13:37:47 +02:00 Compare
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from bea92d1ee5 to 554fe02470 2023-04-13 15:03:23 +02:00 Compare
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from 554fe02470 to 5881756a36 2023-04-13 15:27:51 +02:00 Compare
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from 5881756a36 to 14027d52d6 2023-04-13 15:40:28 +02:00 Compare
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from 14027d52d6 to e29e9b2ccb 2023-04-14 13:09:35 +02:00 Compare
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from e29e9b2ccb to a6cf47c069 2023-04-14 18:10:04 +02:00 Compare
bdauvergne force-pushed wip/43479-connecteur-avis-imposition from a6cf47c069 to 1de7845982 2023-04-15 09:36:12 +02:00 Compare
pmarillonnet commented 2023-05-09 14:49:38 +02:00
Owner
Copy Link

Je commence à relire.

Je commence à relire.
pmarillonnet reviewed 2023-05-10 11:02:18 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok, un peu de la bidouille mais j’imagine qu’on a pas le choix si on veut une solution alternative à API particulier et impôts particulier. Rien à redire sur l’approche, je ne vois pas ce qu’on pourrait faire d’autre, mais juste deux petits commentaires sur des choses que je ne comprends pas à la relecture.

Ok, un peu de la bidouille mais j’imagine qu’on a pas le choix si on veut une solution alternative à API particulier et impôts particulier. Rien à redire sur l’approche, je ne vois pas ce qu’on pourrait faire d’autre, mais juste deux petits commentaires sur des choses que je ne comprends pas à la relecture.
passerelle/apps/avis_imposition/models.py
@ -0,0 +174,4 @@
}
def get_field(name, td_contents, tr_length):
pmarillonnet commented 2023-05-10 10:45:59 +02:00
Owner
Copy Link

Pour quelles raisons tr_length peut être amené à varier ?

Pour quelles raisons `tr_length` peut être amené à varier ?
bdauvergne commented 2023-05-10 14:49:11 +02:00
Author
Owner
Copy Link

Le code étant bien vieux je ne sais plus trop, mais du commentaire plus haut, je déduis que je suis tombé sur des cas où il n'y avait qu'une ou 2 lignes pour décrire l'adresse dans le tableau. Dans les tests du projet d'origine je ne trouve que les cas à 2 ou 3 lignes :

Le code étant bien vieux je ne sais plus trop, mais du commentaire plus haut, je déduis que je suis tombé sur des cas où il n'y avait qu'une ou 2 lignes pour décrire l'adresse dans le tableau. Dans les tests du projet d'origine je ne trouve que les cas à 2 ou 3 lignes : * https://github.com/bdauvergne/svair-api/blob/master/test/resources/postHttpResponse.txt * https://github.com/bdauvergne/svair-api/blob/master/test/resources/postHttpResponse2.txt
pmarillonnet commented 2023-05-10 14:52:24 +02:00
Owner
Copy Link

D’ac, tableau à géométrie variable, pas cool de leur part :/
Merci pour la précision.

D’ac, tableau à géométrie variable, pas cool de leur part :/ Merci pour la précision.
passerelle/apps/avis_imposition/models.py
@ -0,0 +448,4 @@
if not data:
# use test vectors...
data = get_fake_data(numero_fiscal=numero_fiscal, reference_avis=reference_avis)
pmarillonnet commented 2023-05-10 10:57:54 +02:00
Owner
Copy Link

Pourquoi on renvoie des données fausses ici ?

Pourquoi on renvoie des données fausses ici ?
bdauvergne commented 2023-05-10 14:31:04 +02:00
Author
Owner
Copy Link

Dans les autres connecteurs du même type il existe des données de test, mais pas pour ce connecteur il n'y a pas de service de recette, les données de test qu'on peut définir servent à cela. En production il n'y en aura pas, ça renverra toujours None.

Dans les autres connecteurs du même type il existe des données de test, mais pas pour ce connecteur il n'y a pas de service de recette, les données de test qu'on peut définir servent à cela. En production il n'y en aura pas, ça renverra toujours None.
pmarillonnet commented 2023-05-10 14:54:02 +02:00
Owner
Copy Link

Dans les autres connecteurs du même type il existe des données de test, mais pas pour ce connecteur il n'y a pas de service de recette, les données de test qu'on peut définir servent à cela. En production il n'y en aura pas, ça renverra toujours None.

Ok, je ne connaissais pas cette pratique. J’espère juste qu’on oubliera pas d’enlever les données de tests lors du report de la config de recette en prod.

> Dans les autres connecteurs du même type il existe des données de test, mais pas pour ce connecteur il n'y a pas de service de recette, les données de test qu'on peut définir servent à cela. En production il n'y en aura pas, ça renverra toujours None. Ok, je ne connaissais pas cette pratique. J’espère juste qu’on oubliera pas d’enlever les données de tests lors du report de la config de recette en prod.
pmarillonnet approved these changes 2023-05-10 14:54:19 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok pour moi.

Ok pour moi.
bdauvergne closed this pull request 2023-05-15 10:29:42 +02:00
All checks were successful
gitea/passerelle/pipeline/head This commit looks good
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#199
No description provided.
Delete Branch "wip/43479-connecteur-avis-imposition"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?