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

pdf: add fill-form endpoint (#73544) #55

Closed
Ghost wants to merge 0 commits from wip/73544-add-fill-form-in-pdf-connector into main
pull from: wip/73544-add-fill-form-in-pdf-connector
merge into: entrouvert:main
Conversation 4 Commits - Files Changed -
Ghost commented 2023-01-24 15:02:13 +01:00
First-time contributor
Copy Link
No description provided.
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from f75df4d511 to 0e1947979d 2023-01-24 15:23:40 +01:00 Compare
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from 0e1947979d to ebe47bee17 2023-01-24 15:24:12 +01:00 Compare
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from ebe47bee17 to 016f348f24 2023-01-24 15:29:59 +01:00 Compare
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from 016f348f24 to a1c3af0330 2023-01-24 17:21:25 +01:00 Compare
Ghost changed title from WIP: pdf: add fill-form endpoint (#73544) to pdf: add fill-form endpoint (#73544) 2023-01-25 15:27:00 +01:00
bdauvergne commented 2023-01-31 17:56:27 +01:00
Owner
Copy Link

Ça pourrait être sympa de rapporter dans la page du connecteur les champs disponibles du PDF via pdftk dump_data_fields_utf8.

Ça pourrait être sympa de rapporter dans la page du connecteur les champs disponibles du PDF via `pdftk dump_data_fields_utf8`.
bdauvergne requested changes 2023-01-31 22:14:31 +01:00
passerelle/apps/pdf/models.py Outdated
@ -73,0 +100,4 @@
upload_to=resource_file_upload_to,
help_text=_('PDF file, used if not input-form in fill-form payload'),
null=True,
blank=True,
bdauvergne commented 2023-01-31 17:49:20 +01:00
Owner
Copy Link

Ajouter un validators=[validate_pdf] avec un bête byte_content.startswith(b'%PDF-1').

Ajouter un `validators=[validate_pdf]` avec un bête `byte_content.startswith(b'%PDF-1')`.
Ghost commented 2023-02-01 18:21:23 +01:00
Author
First-time contributor
Copy Link

Yep ; validateur PDF basique ajouté.

Yep ; validateur PDF basique ajouté.
bdauvergne commented 2023-02-01 16:40:32 +01:00
Owner
Copy Link

Un coup de rebase aussi.

Un coup de rebase aussi.
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from a1c3af0330 to 3a34438296 2023-02-01 18:17:12 +01:00 Compare
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from 3a34438296 to fb7c40c5a1 2023-02-01 18:20:37 +01:00 Compare
Ghost reviewed 2023-02-01 18:23:21 +01:00
Ghost left a comment
Author
First-time contributor
Copy Link

Voici une nouvelle version avec la validation PDF, un peu de nettoyage des tests (plus complets) et un test de lancement réel de pdftk fill-form.

Je n'ai en revanche pas mis en place l'idée (bonne) du « dump_data_fields_utf8 », je sais pas encore trop comment faire. Je pense pour l'instant à une simple option « ?show-fields » sur le endpoint fill-form, qui renverrai de contenu brut du dump_data_fields_utf8 en text/plain. Ca irait ?

Voici une nouvelle version avec la validation PDF, un peu de nettoyage des tests (plus complets) et un test de lancement réel de pdftk fill-form. Je n'ai en revanche pas mis en place l'idée (bonne) du « dump_data_fields_utf8 », je sais pas encore trop comment faire. Je pense pour l'instant à une simple option « ?show-fields » sur le endpoint fill-form, qui renverrai de contenu brut du dump_data_fields_utf8 en text/plain. Ca irait ?
bdauvergne commented 2023-02-01 18:46:22 +01:00
Owner
Copy Link

Ca irait ?

Si ce qui t'embête c'est de parser le résultat, je pense que tu peux te permettre d'afficher ça dans un pre/pre dans le template héritant de service_view.html correspondant, ce sera toujours plus pratique que de devoir appeler un endpoint.

> Ca irait ? Si ce qui t'embête c'est de parser le résultat, je pense que tu peux te permettre d'afficher ça dans un pre/pre dans le template héritant de service_view.html correspondant, ce sera toujours plus pratique que de devoir appeler un endpoint.
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from fb7c40c5a1 to 68543ee5bc 2023-02-12 17:37:10 +01:00 Compare
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from 68543ee5bc to 3ef8bd6633 2023-02-12 23:32:50 +01:00 Compare
Ghost changed target branch from wip/73540-add-pdf-cat to main 2023-02-12 23:35:18 +01:00
Ghost commented 2023-02-12 23:38:26 +01:00
Author
First-time contributor
Copy Link

Version rebasée, avec l'ajout du résultat de dump_data_fields_utf8 dans un onglet, légèrement adapté pour afficher les noms des fields sous la forme de ce qui doit être envoyé dans fill-form (foo.bar -> fields/foo/bar).

Avec ceci je pense qu'on a quelque chose d'utilisable.

Version rebasée, avec l'ajout du résultat de dump_data_fields_utf8 dans un onglet, légèrement adapté pour afficher les noms des fields sous la forme de ce qui doit être envoyé dans fill-form (foo.bar -> fields/foo/bar). Avec ceci je pense qu'on a quelque chose d'utilisable.
Ghost requested review from bdauvergne 2023-02-12 23:38:34 +01:00
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from 3ef8bd6633 to 71abe5737d 2023-02-12 23:49:03 +01:00 Compare
aberriot requested review from aberriot 2023-02-13 10:45:58 +01:00
aberriot approved these changes 2023-02-14 10:26:08 +01:00
aberriot left a comment
Owner
Copy Link

Testé en local avec https://github.com/departement-loire-atlantique/prefill-pdf-connector/tree/master/prefill_pdf/templates, ça fonctionne comme spécifié à mon sens.

Testé en local avec https://github.com/departement-loire-atlantique/prefill-pdf-connector/tree/master/prefill_pdf/templates, ça fonctionne comme spécifié à mon sens.
Ghost force-pushed wip/73544-add-fill-form-in-pdf-connector from 71abe5737d to 0d9e35cc2f 2023-02-14 11:21:56 +01:00 Compare
Ghost closed this pull request 2023-02-14 12:10:43 +01:00
bdauvergne refused to review 2023-05-04 17:22:03 +02:00
Some checks reported errors
gitea-wip/passerelle/pipeline/pr-main This commit looks good
Details
gitea/passerelle/pipeline/head Something is wrong with the build of this commit
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
aberriot
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#55
No description provided.
Delete Branch "wip/73544-add-fill-form-in-pdf-connector"

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?