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

Zusätzliche Spalte "Stück" in der Buchungsliste eines Kontos #83

Closed
gynngr opened this issue May 31, 2013 · 6 comments
Closed

Zusätzliche Spalte "Stück" in der Buchungsliste eines Kontos #83

gynngr opened this issue May 31, 2013 · 6 comments

Comments

@gynngr
Copy link
Contributor

@gynngr gynngr commented May 31, 2013

wäre sehr hilfreich...

image

@simpsus
Copy link
Contributor

@simpsus simpsus commented May 31, 2013

Diese Spalte macht nicht für jeden "Typ" von Buchungen Sinn. Für eine Einlage zb nicht.
Technisch ist das kein großes Problem, aber bei allen nicht (Ver)kauf Buchungen stünde dann da nix drin.
Hier muss der Master (abuchen) sagen, ob er das für sinnvoll hält...

@buchen
Copy link
Owner

@buchen buchen commented May 31, 2013

+1

"Wertpapier" ist ja auch nicht immer gefüllt.

Fragt sich nur: alle? Also: Stück, Einstandskurs, Gebühren?

@gynngr
Copy link
Contributor Author

@gynngr gynngr commented May 31, 2013

oder konfigurierbar wie in der Wertpapierliste...

image

@simpsus
Copy link
Contributor

@simpsus simpsus commented Jun 10, 2013

Ich habe die Minimalversion (Spalte Stück, die leer ist wenn es die Security auch ist) implementiert. Pull request kann gestellt werden.
Für die Luxus Version mit der Auswahl der Spalten:
Bei den Securities ist das über "SecuritiesTable" realisiert. Macht es dann Sinn hierfür (und eigentlich auch für andere ListViews von Transactions einen "TransactionsTable" zu erstellen?
Ein Problem was ich auf Anhieb sehe ist dass die auswählbaren Spalten nicht zwingend für die Transaktionen Sinn machen müssen. Aber das ist nicht sooo schlimm. Das ist nervig beim implementieren, im Programm sind sie dann einfach leer.

@buchen
Copy link
Owner

@buchen buchen commented Jun 10, 2013

Die Auswahl der Spalten funktioniert noch nicht so einfach, wenn man "in-place" editieren können soll. Ich schaue mir den Commit an und übernehme ihn.

@buchen
Copy link
Owner

@buchen buchen commented Jun 14, 2013

Pull request merged; kommt mit nächster Version.

@buchen buchen closed this Jun 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants