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

EVENTS: Ausbildungstage #256

Closed
codez opened this issue Jan 26, 2024 · 7 comments · Fixed by hitobito/hitobito#2472, hitobito/hitobito_youth#52 or hitobito/hitobito_pbs#316
Closed

EVENTS: Ausbildungstage #256

codez opened this issue Jan 26, 2024 · 7 comments · Fixed by hitobito/hitobito#2472, hitobito/hitobito_youth#52 or hitobito/hitobito_pbs#316

Comments

@codez
Copy link
Contributor

codez commented Jan 26, 2024

Qualifikationsart hat neu ein Feld "Erforderliche Ausbildungstage für Verlängerung" / required_training_days (decimal, precision: 5, scale: 2, nullable). Zusätzlicher Hilfetext beim Input: "Mindestens so viele Ausbildungstage sind während der Gültigkeitsdauer zu absolvieren, um bestehende Qualifikationen zu verlängern. Diese können auch über mehrere Kurse aufgeteilt werden. Leer lassen, um Ausbildungstage nicht zu berücksichtigen und Qualifikationen unmittelbar bei jedem entsprechenden Kursbesuch zu verlängern."

Kurs hat neu ein Feld "Ausbildungstage" / training_days (decimal, precision: 5, scale: 2, nullable). Dieses ist bereits im Youth Wagon vorhanden und soll in den Core aufgenommen/verschoben werden, inklusive der Anzeige auf der Infoseite des Events und im Formular. Letzteres wird somit im Youth Wagon entfernt. Übrige Funktionalität, welche bisher im Youth Wagon ist und dieses Feld betrifft, bleibt dort.

@codez
Copy link
Contributor Author

codez commented Jan 26, 2024

@ThomasEllenberger / @mtnstar seht ihr das im Core, Youth, oder Sac? Wahrscheinlich könnte das für alle interessant sein (Logik folgt in weitern Tickets: #257, #262).

@mtnstar
Copy link
Contributor

mtnstar commented Jan 29, 2024

aus technischer Sicht sehe ich keine Probleme im Core realisieren.
Fachlich kann vermutlich @ThomasEllenberger das besser beurteilen

@ThomasEllenberger
Copy link

Wir hatten auch schon Anfragen welche in diese Richtung giengen, denk desshalb dass das im Core umgesetzt werden kann.
Zwingend notwendig ist dabei sicher der Hilfetext.

@codez
Copy link
Contributor Author

codez commented Jan 30, 2024

Stellt sich noch die Frage, ob wir #261, falls wir das bauen, dann auch in den Core nehmen wollen?

@codez codez changed the title EVENTS: Fortbildungsstunden EVENTS: Ausbildungstage Jan 31, 2024
@codez codez added the Core label Feb 2, 2024
@sykesonrocks sykesonrocks assigned codez and unassigned sykesonrocks Mar 1, 2024
@codez codez assigned codez and unassigned codez Mar 1, 2024
codez added a commit to hitobito/hitobito that referenced this issue Mar 8, 2024
codez added a commit to hitobito/hitobito_youth that referenced this issue Mar 8, 2024
codez added a commit to hitobito/hitobito_pbs that referenced this issue Mar 8, 2024
amaierhofer pushed a commit to hitobito/hitobito that referenced this issue Mar 12, 2024
amaierhofer pushed a commit to hitobito/hitobito that referenced this issue Mar 12, 2024
amaierhofer pushed a commit to hitobito/hitobito_youth that referenced this issue Mar 12, 2024
amaierhofer pushed a commit to hitobito/hitobito_pbs that referenced this issue Mar 12, 2024
@sykesonrocks
Copy link
Collaborator

@codez Nachfolgend meine Testergebnisse.

  • Qualifikationsart hat neu ein Feld "Erforderliche Ausbildungstage für Verlängerung" und der Hilfetext wird wie oben spezifiziert dargestellt. Somit Ok.
  • Das neue Feld auf dem Kurs, "Ausbildungstage", kann ich aktuell nicht testen. Wenn ich versuche einen neuen Kurs anzulegen, erhalte ich folgende Fehlermeldung:
    image

@codez codez removed their assignment Mar 19, 2024
@codez
Copy link
Contributor Author

codez commented Mar 19, 2024

@sykesonrocks Jetzt sollte alles funktionieren

@sykesonrocks
Copy link
Collaborator

@codez Danke. Habe es soeben getestet und sehe auf dem Kurs das neu eingeführte Feld "Ausbildungstage", somit OK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment