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

ISI-837 Auslösen von Sobon Berechnung und Auswahl des Sobon wichtigem Fördermix #149

Merged
merged 192 commits into from
Mar 18, 2024

Conversation

TiagoKocevar
Copy link
Contributor

@TiagoKocevar TiagoKocevar commented Mar 6, 2024

Description

Neue Attribute isASobonBerechnung (boolean) und sobonFoerdermix (Foerdermix).

Wenn isASobonBerechnung wahr ist soll eine SobonBerechnung ausgeführt werden. Aber nur wenn auch der sobonFoerdermix gesetzt wurde.

Dann wird der sobonFoerdermix als Basis hergenommen für die Berechnung.

Validation hinzugefügt das sobonFoerdermix einen Wert haben muss wenn isASobonBerechnung wahr ist.

Außerdem Validation abgeändert bei der Fördermixklasse da bei der Abfragevariante die Felder sobonFoerdermix und isASobonBerechnung null sein dürfen.

Reference

Issues #837

banzuu and others added 30 commits November 10, 2023 11:51
* Umlegung Review Änderungen implementiert

* Precision angepasst

* Kommentar entfernt

* Testing added

* Kommentar entfernt

* Logik bezüglich Gültig ab gefixt

* Repository Method fixed
@TiagoKocevar TiagoKocevar force-pushed the ISI-837-SoBoN-Ursaechlicher-Fördermix-Auslösen branch from 640c606 to 3d7078e Compare March 8, 2024 12:39
@TiagoKocevar TiagoKocevar changed the title DRAFT: ISI 837 ISI-837 Auslösen von Sobon Berechnung und Auswahl des Sobon wichtigem Fördermix Mar 8, 2024
@boal boal self-requested a review March 11, 2024 06:29
@TiagoKocevar TiagoKocevar self-assigned this Mar 11, 2024
boal
boal previously approved these changes Mar 13, 2024
@boal boal merged commit 8cd7e7f into dev Mar 18, 2024
3 checks passed
@boal boal deleted the ISI-837-SoBoN-Ursaechlicher-Fördermix-Auslösen branch March 18, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants