-
Notifications
You must be signed in to change notification settings - Fork 592
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
dividend per share proposal #689
dividend per share proposal #689
Conversation
…agsgutschriftErtraegnisgutschrift.txt Rename fielname and avoid umlaut for interoperatability
…ragsgutschriftErtraegnisgutschrift2.txt Rename filename and avoid umlaut for interoperatability
Rename filename and avoid umlaut for interoperatability
…tragEingang.txt Rename filename and avoid umlaut for interoperatability
… to abort in ubuntu based Docker container
Calculate it based on GrossAmount or calculate GrossAmount based on dividend
…Plugin" This reverts commit 2480fcb.
Die letzten 3 Commits (2480fcb, ee973bb, 2aeda6c) gehören eigentlich nicht zu der Change. Vielleicht könntest Du die komplette Change (nach den Änderungen unten) per Squash zusammenführen und noch mal pushen? Dann sind mir noch zwei Kleinigkeiten aufgefallen: Erstens wird die "Dividende pro Stück" nicht gesetzt wenn man die Buchung editieren möchte (da ist aber noch ein TODO im Code). Zweitens kann die Anzahl der Stücke auch 0 sein. Das ist ein bisschen historisch bedingt. Aber der Code sollte damit umgehen können. Aktuell steht dann da "-0,01". |
…no shares are present (based on command @abuchen in portfolio-performance#689)
Calculate it based on GrossAmount or calculate GrossAmount based on dividend Introducing new label for dividend per share field recalc dividendPerShare also when editing, set dividend to 0 in case no shares are present (based on command @abuchen in portfolio-performance#689)
…ng/portfolio into cmaoling-dividendPerShare
…Plugin" This reverts commit 2480fcb.
…agsgutschriftErtraegnisgutschrift.txt Rename fielname and avoid umlaut for interoperatability
…ragsgutschriftErtraegnisgutschrift2.txt Rename filename and avoid umlaut for interoperatability
Rename filename and avoid umlaut for interoperatability
…tragEingang.txt Rename filename and avoid umlaut for interoperatability
… to abort in ubuntu based Docker container
…ng/portfolio into cmaoling-dividendPerShare
Habe Deine Änderung eingebaut - denke ich. |
Hast Du dich "vergittet"? Kenne ich... Im Prinzip ist das ja der Commit d53ff0f. Den habe ich mir geholt und auf den Master gesetzt. Jetzt ist es drin. |
@cmaoling Habe ich nicht den kompletten / richtigen Change mit reingenommen? siehe auch hier: https://forum.portfolio-performance.info/t/fehler-bei-berechnung-in-maske-bei-dividenden/461 Es scheint wir müssen - zumindest dem Dialog - noch ein paar Nachkommastellen spendieren. Vor dem WE komme ich zu nix. Falls Du Zeit hättest? Ich habe auch an anderen Stellen schon mit BigDecimal gearbeitet (nur zur Rechnung). |
@buchen. Ok, Ich schaue rein. |
…vidend w/ increased accurency of 7 digit, rather than 2
initial implementation to address #688