Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0] account_invoice_import_simple_pdf: works with newer PyMuPDF versions & add pypdf backend #836

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

alexis-via
Copy link
Contributor

works with newer PyMuPDF
Add pypdf text extraction backend
Update README for pypdf, and also for the fact that PyMuPDF is now easy to install (to additional binary/lib required)
Remove embedded XML file from PDF test file tests/pdf/akretion_france-test.pdf ! (otherwise, it is parsed by account_invoice_import_facturx if installed !)

@alexis-via alexis-via force-pushed the 14-simple_pdf-pymupdf_update branch from b3e402d to dffa5e6 Compare October 20, 2023 18:47
Add pypdf text extraction backend
Update README for pypdf, and also for the fact that PyMuPDF is now easy to install
Remove embedded XML file from PDF test file tests/pdf/akretion_france-test.pdf !
@alexis-via alexis-via force-pushed the 14-simple_pdf-pymupdf_update branch from dffa5e6 to 9de1261 Compare October 20, 2023 18:57
@alexis-via
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-836-by-alexis-via-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b985b6e into OCA:14.0 Oct 24, 2023
5 of 6 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 80bf234. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants