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

NEW Quadratus export with attachments in accountancy export #23209

Conversation

lvessiller-opendsi
Copy link
Contributor

NEW Quadratus export with attachments in accountancy export

In accountancy bookeeping list :
image

Add a checkbox to export documents (only for Quadratus export format) :
image

When this checkbox is "checked" : a new archive file is created with the exported file and all documents linked to accountancy bookeeping lines:
image

In this example, the PDF file is named "00000001.pdf" (piece number in accountancy bookeeping table) and the content provide from reference of the accountancy bookeeping line.

Without this checkbox, the previous behavior is kept and we download one file (TXT or CSV).

Copy link
Member

@aspangaro aspangaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok for me. Need tester like @josett225

@eldy eldy added the PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do) label Dec 22, 2022
@lvessiller-opendsi
Copy link
Contributor Author

@eldy
All conflicts resolved

@eldy eldy merged commit 0feac89 into Dolibarr:develop Jan 4, 2023
@lvessiller-opendsi lvessiller-opendsi deleted the new-accountancy-export-quadatus-with-archive branch January 4, 2023 07:36
@josett225
Copy link
Contributor

Hi @aspangaro
Quadratus Tests are starting next week.
Jose

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR to fix - See feedback in comments PR needs to be fixed to be integrated (some comments should describes the fix to do)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants