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

Feature chartview #489

Closed
wants to merge 5 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@hjbflyer
Contributor

hjbflyer commented Apr 8, 2016

Die TimelineChart wird jetzt in einer neuen Klasse als eigene View dargestellt. Deshalb ist der ganze
ViewHandling Code aus der Klasse SecuritiesListView in die TimlineChartView gewandert. Dieser
Code konnte damit leicht in die Klasse SecuritiesPerfomanceView aufgerufen werden.

Alle Änderungen sind in dem letzten Commit

@buchen buchen added the enhancement label Apr 9, 2016

@buchen buchen added this to the 2016 Q1 Short-List milestone Apr 9, 2016

@buchen

This comment has been minimized.

Owner

buchen commented Apr 9, 2016

hi @hjbflyer

Vielen Dank für den Pull request! Es macht absolut Sinn das Kursdiagramm auch bei der Wertpapier Performance zu zeigen.

Ich werde mir die Commits aus dem PR holen (das gitignore und das parsen des Datums habe ich ja schon im master branch gemacht). Kein Problem.

Ich habe überlegt: vielleicht macht es auch Sinn, den Zeitraum des Charts auf den Berichtszeitraum zu setzen. Man kann das ja über die Buttons jederzeit ändern, aber initial wäre es identisch zu dem Berichtszeitraum für den man sich die obigen Daten anschaut. Was meinst Du?

buchen added a commit that referenced this pull request Apr 9, 2016

Extract security chart to be also used in security performance view
Issue: #489
Signed-off-by: Hans-J. Brede <hjb@hjbrede.de>
[formatting, renamed chart class]
Signed-off-by: Andreas Buchen <andreas.buchen@gmail.com>

buchen added a commit that referenced this pull request Apr 9, 2016

Fix NPE if no security is selected in security performance view
Issue: #489
Signed-off-by: Hans-J. Brede <hjb@hjbrede.de>
[formatting]
Signed-off-by: Andreas Buchen <andreas.buchen@gmail.com>
@buchen

This comment has been minimized.

Owner

buchen commented Apr 9, 2016

Ich habe die Changes jetzt per Cherry-pick in den master übernommen.

Wenn Du weitere Änderungen machen willst, dann ist es aus meiner Erfahrung am einfachsten, wenn Du einen neuen Feature Branch auf meinem master branch machst (der müsste bei Dir im Tool als "remote" eingetragen sein.

Ich verwende SourceTree als Git Client. Da kann ich dann die Remotes eintragen (wie Du siehst habe ich Dein Repository eingetragen um mir die Commits zu holen) und dann per Rechtsklick -> Checkout einen neuen Branch anlegen:
bildschirmfoto 2016-04-09 um 08 56 13

@buchen buchen closed this Apr 9, 2016

@hjbflyer

This comment has been minimized.

Contributor

hjbflyer commented Apr 9, 2016

Ja, finde ich gut.

Am 09.04.2016 um 08:43 schrieb Andreas Buchen notifications@github.com:

hi @hjbflyer https://github.com/hjbflyer
Vielen Dank für den Pull request! Es macht absolut Sinn das Kursdiagramm auch bei der Wertpapier Performance zu zeigen.

Ich werde mir die Commits aus dem PR holen (das gitignore und das parsen des Datums habe ich ja schon im master branch gemacht). Kein Problem.

Ich habe überlegt: vielleicht macht es auch Sinn, den Zeitraum des Charts auf den Berichtszeitraum zu setzen. Man kann das ja über die Buttons jederzeit ändern, aber initial wäre es identisch zu dem Berichtszeitraum für den man sich die obigen Daten anschaut. Was meinst Du?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub #489 (comment)

@hjbflyer hjbflyer deleted the hjbflyer:feature-chartview branch Apr 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment