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

Berichte Buchungsliste Export aller Konten als PDF (ZIP) ermöglichen #265

Merged
merged 4 commits into from
Mar 5, 2024

Conversation

rebootl
Copy link
Member

@rebootl rebootl commented Feb 29, 2024

Analog zum CSV export.

Changelog aktualisiert.

Analog zum CSV export.

Changelog aktualisiert.
@bblessmann
Copy link
Member

Hi Cem,
dieser PR sieht soweit gut aus. Ich habe aber noch Probleme gefunden, die wohl auch vor dem PDF-Export vorhanden sind. Schicke ich Dir per Mail bzw. in die Mailingliste.

@rebootl
Copy link
Member Author

rebootl commented Mar 5, 2024

Okay danke, email gesehen, ich schau mir das nachher an.

rebootl added 3 commits March 5, 2024 14:06
Der export als chart funktioniert hier nicht,
und hat hier vermutlich auch nur bedingt nutzen,
daher deaktiviert.
Titel muss escaped werden da Sonderzeichen in der Kontonummer/
Beschreibung möglich sind.
Dateinamen müssen für das ZIP file bereinigt werden,
da Sonderzeichen in der Kontonummer möglich sind.
@rebootl
Copy link
Member Author

rebootl commented Mar 5, 2024

Okay, die Sachen aus der Mail sollten mit den obigen commits behoben sein.

Für das bereinigen der Dateinamen habe ich eine regex verwendet:

my $sanitized_accno = $account->{accno} =~ s/[^A-Za-z0-9\-\.\_\ ]/_/gr;

Falls jemand eine bessere Idee hat gerne melden.

@bblessmann bblessmann merged commit e7b1364 into master Mar 5, 2024
1 check passed
@bblessmann bblessmann deleted the 20240229-feat-list-transactions-export-pdf branch March 5, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants