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

Weiterbildung hinzufügen ohne Datumsrange nicht mehr möglich #450

Closed
hupf opened this issue May 10, 2021 · 5 comments
Closed

Weiterbildung hinzufügen ohne Datumsrange nicht mehr möglich #450

hupf opened this issue May 10, 2021 · 5 comments
Assignees
Labels
plan needs to be planned

Comments

@hupf
Copy link
Member

hupf commented May 10, 2021

Erwartetes Verhalten

  • Ich öffne das CV einer Person
  • Ich klicke unter Weiterbildungen auf Weiterbildung hinzufügen
  • Ich fülle das Formular wiefolgt aus:
    image
  • Ich klicke auf Speichern.
  • Es sollte ein Eintrag hinzugefügt werden mit dem Wert "01.2020" in der ersten Spalte.

Effektives Verhalten

Nach dem Hinzufügen wird in der Tabelle immer ein Range angezeigt:
image

Beim Bearbeiten dieses Eintrages wird das Bis auf "heute" gesetzt (die Option "-" lässt sich nicht auswählen, das Select springt zurück auf "heute"):

image

Ich weiss, dass es mal korrekt funktionierte, da ich einen Eintrag habe, bei dem nur ein Jahr angezeigt wird.

@Robin481 Robin481 added the plan needs to be planned label May 10, 2021
@Robin481
Copy link
Member

Robin481 commented May 14, 2021

Hoi @hupf
Ich habe mir das nun angeschaut.
Du kannst deine gewünschte Funktionalität erreichen in dem du in den Feldern von und bis die gleichen Werte einträgst.
Dann wird es dir nur noch den Wert 01.2020 anzeigen.
Ich bin einverstanden das dies nicht ganz Intuitiv ist.
Die Idee dahinter ist das immer entweder der Eintrag noch läuft (also - heute) oder das eine Range eingetragen ist.

@hupf
Copy link
Member Author

hupf commented May 17, 2021

Hey @Robin481
Danke fürs Abklären. Effektiv, so konnte ich es eingeben.

Die Idee dahinter ist das immer entweder der Eintrag noch läuft (also - heute) oder das eine Range eingetragen ist.

Aus der Benutzerperspektive ist es tatsächlich nicht sehr intuitiv, dass es immer ein Range sein muss. Insb. da viele Weiterbildungen einfach eine 1-2 Wöchige Ausbildung mit Zertifikat sind. Da ist relevant, wann das Zertifikat gemacht wurde.

Ich denke, wenn man beim Bis die leere Auswahl (-) wählen könnte, wäre das easy (intern könntet ihr ja immer noch den Range mit gleichem Datum speichern).

@Robin481
Copy link
Member

Robin481 commented Mar 7, 2023

TODO:

  • Allow - selection in dropdown (or no end or something, open to fancy wording ideas. We could also just add a checkbox along the lines of do not set a range or something.)
  • This behaves the same way as setting the end date as the same value as the from date
  • When a daterange that has been set like this is edited again it can show the to date as we saved it in the database and doesn't need to display - or kein Ende (this should make things easier)
  • Write a little test
  • Smoke test

@MarcEgliP
Copy link
Collaborator

MarcEgliP commented Mar 20, 2023

Vorgehen:

  • Form öffnen
  • Komponent ausfindig machen
  • Option "-" verfügbar machen, mit value des ersten Fields (andere Option gefunden)
  • Komponent testen
  • Smoke testen
  • Code formatting, ungebrauchter Code löschen
  • Feature pushen & Pullrequest stellen

@MarcEgliP MarcEgliP self-assigned this Mar 20, 2023
@Robin481
Copy link
Member

Robin481 commented Jun 2, 2023

Fixed with the new release, thanks for the issue!

@Robin481 Robin481 closed this as completed Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plan needs to be planned
Projects
None yet
Development

No branches or pull requests

3 participants