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

Planner: publish overrun duration #13126

Merged
merged 3 commits into from
Mar 28, 2024
Merged

Planner: publish overrun duration #13126

merged 3 commits into from
Mar 28, 2024

Conversation

andig
Copy link
Member

@andig andig commented Mar 24, 2024

Fix #12703

TODO

  • UI

@andig andig added the enhancement New feature or request label Mar 24, 2024
@naltatis
Copy link
Member

Oh, das ist dann ja noch mal anders. Lass uns die Datengrundlage für die Warnung (geschätzte Endzeit) gleich halten.
Ich hab nichts dagegen den Overrun als duration zu übertragen. Aber dann müssen wir das in den /plan[/preview] endpoints auch so machen. Hier lese ich aktuell die Endzeit und vergleiche die mit der angefragten Zeit.

@andig andig added the prio Priority label Mar 26, 2024
@naltatis
Copy link
Member

@andig ich hab jetzt mal pragmatisch die UI Anpassungen für den Vorschlag gemacht. Nun wird immer auf max 1 Minute Toleranz geprüft.

@andig andig merged commit 6446d92 into master Mar 28, 2024
6 checks passed
@andig andig deleted the fix/plan-overrun branch March 28, 2024 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request prio Priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ladeplanung: "Zielzeit nicht erreichbar" (Anzeige- oder Berechnungsproblem)
2 participants