-
Notifications
You must be signed in to change notification settings - Fork 638
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
Enhancement dividend as event referring to #697 #699
Enhancement dividend as event referring to #697 #699
Conversation
…ndle Dividends. Data of such events stored including Currency
Wenn es Dir darum geht, die Dividendenzahlungen ebenfalls mit ins Chart aufzunehmen, dann wäre es mir lieber die in der Klasse "SecuritiesChart" auszulesen und direkt "addMarkerLines" aufzurufen. Warum vorher einen SecurityEvent anlegen? In einem zweien Schritt sollten wir überlegen, ob man das Ein/Ausblendbar machen kann. Wenn ich Diagramme wie dieses hier sehe, dann ist das schon nicht mehr hilfreich. In die Klasse SecurityEvent gehört das so (zumindest die konkrete Zahl) nicht rein. Erstens wird das mit in dem XML persistiert. Damit hätte man aber die Dividendenbuchung mehrfach - was wenn man die Buchung editiert. Zweitens sollten die SecurityEvents unabhängig von einer Buchung. Also das technische Ereignis "Dividende" kann ich mir so vorstellen. Dann aber auch, wenn ich das Wertpapier gar nicht besitze und vielleicht die Performance inklusive der potentiellen Dividendenzahlungen vergleichen möchte. Auf Basis dieses "Dividenden Events" könnte man dann vorschlagen eine Dividendenbuchung zu erfassen. Oder ähnliches. Das ist aber eine grössere Geschichte. Von Yahoo Finance könnte man diese Dividenden-Events bekommen. |
Abend. Das Ein- und Ausschalten kann man sich überlegen. Hatte auch schon gedacht, das konfigurierbar zu machen. Wollte aber erstmal diese Änderung einbringen.... Ok, erwischt. Die Persistent im XML hat mich sogar sehr erfreut. Dein dritter Absatz beschreibt ziemlich genau was ich am Ende einbauen will. 🥇 |
Im Chart sind die natürlich noch nicht, aber genauso wie ich aus den (Ver)Kaufsbuchungen marker lines generiere, könnte man ja aus den Dividendenbuchungen ebenfalls marker lines generieren - eben aus den AccountTransactions. In der Klasse Security gibt es auch eine Methode getTransactions mit der man alle zu diesem Wertpapier gehörenden Buchungen bekommt.
Deine Idee ist auch gut. 😄 Ariva könnte vielleicht schwierig werden wegen der Nutzungsbedingungen - die habe ich jetzt allerdings nicht im Kopf. Aber bei Yahoo Finance gibt es die Informationen + PP ruft ja schon Kurse ab. Man müsste mit einem weitere Flag die Dividendeninformationen abfragen und verarbeiten. Der Kurse-Update hat über die Jahre etwas technical debt angesammelt, darum ist das vielleicht nicht ganz gerade aus. Wenn man das hat, kann man in weiteren Schritten:
... In diesem Change würde ich jetzt erst mal die Dividenden Marker Lines unabhängig von SecurityEvent einbauen. |
Abend. Um hier doppelte Arbeit zu vermeiden eine sprachliche Klarstellung: |
Guter Punkt - da war ich unsauber. Ich mache jetzt erst mal nichts (bin sowieso mit PDF Dokumenten und anderen GUI Bugs beschäftigt) |
…ed on AccountTransactions also
@buchen: "In diesem Change würde ich jetzt erst mal die Dividenden Marker Lines unabhängig von SecurityEvent einbauen." - Done. |
Hi @cmaoling, ich habe mir per Cherry pick den letzten Change geholt und in den master gemergt. Die Änderungen am SecurityEvent habe ich erst mal nicht mit aufgenommen - wie oben schon gesagt, habe ich das Problem mit der aktuellen Implementierung, dass es die Dividendenbuchung doppelt persistiert und nicht editierter ist, etc. Abgesehen fehlt ja noch mehr drumherum das man mit den erweiterten SecurityEvent was angefangen kann --> separater Branch. Auf die Change habe ich jetzt ein kleines Konfigurationsmenü gesetzt. Dann kann man wählen ob man die Dividenden sehen möchte oder nicht. Gerade bei meinen amerikanischen Wertpapiere, die einmal im Quartal ausschütten, können das ganz schön viele Linien sein. |
…ed on AccountTransactions also
No description provided.