search

KarelZe / DHBW_MWI

MWI Projekt :mortar_board:
0 stars 0 forks source link

Sammel Issue Perioden Pflegen #36

Closed Franzhose closed 5 years ago

Franzhose commented 5 years ago

Mir sind mehrere Dinge aufgefallen:

  1. Aktiver Tab wird bei Laden der Seite nicht gehighlighted. ☑️
  2. Periode kann nach abschließen noch verändert werden. Ist es möglich den Tab auszugrauen?☑️
  3. TransaktionsartId hat wert null bei festgeldverzinsung in der Buchungstabelle // erzeugt nullpointerexceptions bei wertpapier_verkaufen ☑️
  4. Prüfen, dass Feld Manueller Kurs optional zu machen.:x:
  5. Prüfen, ob berechneter Kurs angezeigt werden kann. ❓
  6. Sicherstellen, dass Anwender einen Spread erfasst. Wenn Anwender keinen Wert explizit in den Spread eingibt, wird bei Spread null in die Datenbank eingetragen, das führt dazu, dass in Zeile 34 vom FloatingRateNoteModel eine NullPointerException fliegt. ☑️
  7. Periode kann nach erneutem Aufruf wieder und wieder gepflegt werden. :x:
  8. Inkonsistentes Layout.☑️
  9. Überprüfung, ob Bewertungszeitpunkt für Floater und ETF korrekt ist.☑️
  10. Manueller Kurs von z. B. 101 als 101 und nicht als 1.01 speichern. ☑️
KarelZe commented 5 years ago

Zu 1. Aktiver Tab wird nun beim Laden im DHBW Rot hevorgehoben. Gefixt mit d9c2079e88486d25daea6b765542dd374fd0b9bb und 0728b3a1446353c00ba1461724029296f01ca88e. Zu 2. Gelöst mit 104c651a90d3ce152f699635af1dd9e1324d92d6. Zu 3. Ist gelöst durch @jackderblaueaffe mit 613c36aabc68d078ca686ac438ed65f90bcbaeda und 2723af538a4d4d78102c35362f55fa4e84ec7bdd. Zu 4. Habe mich dagegen entschieden, macht die Ansicht nur noch unübersichtlicher. Zu 5. Werde ich anbetracht der Zeit streichen, da auch keine Änderungsmöglichkeit für den Dannenmayr besteht. Außer ich finde noch Zeit, dann setze ich es um. Zu 6. Ist gelöst mit b75ee809a7e9d30efadb494967acdd92fabd8a3a. Ich habe mich dazu entschieden, bei Anlage einer neuen Periode den Spread für Anleihen mit 0 zu setzen. Dies ermöglicht, dass der Spread weiterhin nullable ist. Gleichzeitig ist sichergestellt, dass alle Anleihen einen Spread haben. Dies ist zugleich die einzige Stelle, wo Kurse angelegt werden, weshalb ich es dort aufgenommen habe. Jetzt informiert auch eine Fehlermeldung über einen ungültigen Spread bzw. alle Falscheingaben. Zu 7. Ist redundant mit 2. Zu 8. Gefixt mit d1d29655cb39bf4cd2d4dc6d2ffb25da668f9131. Zu 9. Gelöst mit d1d29655cb39bf4cd2d4dc6d2ffb25da668f9131. Zu 10. Ist gefixt mit 86382a3d497764ca68f8e7df09faf6bede16c71c. @Franzhose @jackderblaueaffe Passt so für euch?

KarelZe commented 5 years ago

Werde Issue schließen. Für Dokumentation hinsichtlich Bearbeitung siehe https://github.com/KarelZe/DHBW_MWI/issues/36#issuecomment-508924690. @Franzhose @jackderblaueaffe @MrApplebyte bitte meldet euch, falls euch beim Testen was auffällt und macht den Issue wieder auf. Herzlichen Dank. 🎉