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

Modify ebase PDF-Importer to support new fee format #2630

Merged
merged 2 commits into from Jan 11, 2022
Merged

Modify ebase PDF-Importer to support new fee format #2630

merged 2 commits into from Jan 11, 2022

Conversation

Nirus2000
Copy link
Member

https://forum.portfolio-performance.info/t/pdf-import-fuer-ebase/7204/57
Fix fee transaction for Depotführungsentgelt
Add more tests to check if the PDF importer works correctly

Hallo @buchen
bei ebase werden die Depotführungsgebühren mit einem Verkauf von Anteilen
abgegolten. (siehe hier) Hier ergab sich folgendes Problem, das z.B. der PDF-Debug "Umsatzabrechnung12.txt"
unvollständig war, da hier der Verkauf der Anteile fehlten. Ergo hatte das zufolge, dass es dazu kam,
dass eventuell doppelt die Gebühren gebucht wurden.
Mit dem ersetzten PDF-Debug aus dem Forum (siehe oben), sieht man, wie die Buchungen korrekt laufen bei Ebase.
Daher habe ich den PDF-Debug "Umsatzabrechnung12.txt" gegen den PDF-Debug aus dem Forum ersetzt und die
fehlerhaften Buchungen entfernt. DIes betraf lediglich "Umsatzabrechnung14.txt".

Gruß Alex

@buchen buchen merged commit 53c3cde into portfolio-performance:master Jan 11, 2022
@Nirus2000 Nirus2000 deleted the Modify-ebase-PDF-Importer-to-support-new-fee-format branch January 12, 2022 16:08
@christen90
Copy link
Contributor

Genial. Wollt mich gerade dransetzen. Dein Vorschlag im Pull Request hat jedoch meine PDF gefressen.

Dankeschön

@Nirus2000
Copy link
Member Author

@christen90
super, dass dies geklappt hat :-)

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.

None yet

3 participants