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

Nur EvaluationUntil benutzen und nicht EvaluationUntil und HasEvaluationStarted #471

Closed
schefbi opened this issue Nov 9, 2022 · 3 comments
Labels
bug Something isn't working module-Anlassliste Feature für das Modul "Liste der Anlässe"

Comments

@schefbi
Copy link
Contributor

schefbi commented Nov 9, 2022

In EvaluationUntil wird das feld HasEvaluationStarted miteinbezogen. Wenn das Datum 2022-01-01T00:00:00 dann kann bis und mit 2022-01-01 an diesem Tag bewertet werden und nicht nur bis Mitternacht. Dieses Feld EvaluationUntil soll nur für die Anzeige genutzt werden.

@schefbi schefbi self-assigned this Nov 9, 2022
@schefbi schefbi added bug Something isn't working module-Anlassliste Feature für das Modul "Liste der Anlässe" labels Nov 9, 2022
@schefbi schefbi removed their assignment Nov 10, 2022
@hupf
Copy link
Collaborator

hupf commented Nov 10, 2022

@schefbi Bist du da dran, oder ist die Idee, dass wir dieses Issue umsetzen? Wenn Letzteres müssten wir etwas mehr Informationen haben. Wir haben versucht zu verstehen was du gemeint haben könntest:

  • Geht es um die zweite Condition «Bewertung bis EvaluationStatusRef.EvaluationUntil» aus Liste der Anlässe: Links #427? Meinst du dass konkret beim EvaluationStatusRef.EvaluationUntil >= Date.Now nur der Tag berücksichtigt werden soll, und nicht die Uhrzeit?
  • Bedeutet «Dieses Feld EvaluationUntil soll nur für die Anzeige genutzt werden.», dass es einen Change bei der Anzeige braucht, oder stimmt es so wie es aktuell umgesetzt ist?
  • Kannst du uns ein Beispiel geben/konstruieren wo dies in den Daten so ist?

@schefbi
Copy link
Contributor Author

schefbi commented Nov 10, 2022

@hupf Ich habe das gestern bereits umgesetzt, aber ich habe meine Entwicklungsumgebung nicht mehr zu laufen gebracht. Ich habe es dennoch nun einmal Commited in branch bug/471-EvaluationUntil-dont-check-current-date.

Du kannst es mit dem User [email protected] testen. Ich habe ein Grenzwert Test erfasst. Die markierten 2 müssen bewertbar sein.
image

@hupf
Copy link
Collaborator

hupf commented Nov 14, 2022

@schefbi Ich habe einen PR erstellt und kommentiert: #477 (review)

@hupf hupf assigned schefbi and unassigned hupf Nov 21, 2022
schefbi added a commit that referenced this issue Nov 22, 2022
schefbi added a commit that referenced this issue Nov 22, 2022
…-check-current-date

Fix EvaluationUntil don't check current date #471
schefbi added a commit that referenced this issue Nov 22, 2022
…-check-current-date

fix status "Überprüfung Zwischenbeurteilung" update tests #471
schefbi added a commit that referenced this issue Nov 22, 2022
@schefbi schefbi closed this as completed Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working module-Anlassliste Feature für das Modul "Liste der Anlässe"
Projects
None yet
Development

No branches or pull requests

2 participants