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

Holiday feature fertigstellen #1013

Closed
6 of 12 tasks
dirkdrutschmann opened this issue Apr 9, 2022 · 5 comments · Fixed by #1296 or #1402
Closed
6 of 12 tasks

Holiday feature fertigstellen #1013

dirkdrutschmann opened this issue Apr 9, 2022 · 5 comments · Fixed by #1296 or #1402
Assignees
Labels
enhancement New feature or request

Comments

@dirkdrutschmann
Copy link
Contributor

dirkdrutschmann commented Apr 9, 2022

branch experiment/timeframe-multiselect

  • Mehrfachauswahl von Locations/Items in Holiday Timeframe
  • Mehrfachauswahl von Locations in Buchbaren Timeframe inkl. Validierung
  • Möglichkeit von Eingabge von Feiertage (@hansmorb ist dran)
  • Feiertagsfunktion per Kategorie (tbd @chriwen: lass uns da mal noch einmal drüber sprechen, wie das funktionieren soll)
  • ?

branch feature/holidays

  • API Curl
  • OptionsPage Timeframe => Manuelles Eingabe Feld nur sichtbar, wenn Repetition->Manual und Type->Bookable||Holiday ausgewählt ist.
  • Import der API_Daten ins Manuelle Feld => Feld nur sichtbar bei Repetition->Manual und Type -> 'Holiday'
  • Logik was passiert mit den Daten => String To Timeframe
  • Timeframevalidierung für manuelle Felder erweitern
  • Buchungsvalidierung für manuelle Felder erweitern
  • Description und angezeigte Namen der Felder auf der Options-Seite
@dirkdrutschmann dirkdrutschmann added the enhancement New feature or request label Apr 9, 2022
@hansmorb hansmorb added this to Backlog in commonsbooking (> v2.0.0) via automation Apr 9, 2022
@hansmorb hansmorb moved this from Backlog to In progress in commonsbooking (> v2.0.0) Apr 9, 2022
@markus-mw markus-mw self-assigned this Apr 10, 2022
@chriwen
Copy link
Member

chriwen commented Apr 21, 2022

Anforderungen aus Dopplung #9210

Übertrag aus Anforderung Hannes:

Alternativ zur Auswahl von Startdatum und Enddatum gibt es beim Anlegen eines Zeitrahmens die Möglichkeit, einzelne Tage kommagetrennt in ein Textfeld einzugeben.

Checkbox "Freie Eingabe von Tagen" unter dem Enddatum (default: nicht aktiv)
bei Aktivierung der Checkbox: Ausblenden von Startdatum und Enddatum, Einblenden eines Textfeld
Übernahme aller im Textfeld aufgelisteten Daten in den Zeitrahmen. Format z.B. (05.05.2021,06.03.2022,usw.)
Erklärtext zum Datumsformat inkl. Beispiele unter dem Textfeld
Vorteile:
Diese Option kann dann genutzt werden, um spezielle Datumskombinationen in nur einen Zeitrahmen einzugeben, anstatt viele Zeitrahmen nutzen zu müssen. Weiterhin ist es möglich, eine einmal zusammengestellte Kombination auch in weitere Zeitrahmen anderer Artikel/Stations-Kombinationen zu kopieren.

Nutzung z.B. für

Feiertage
Regelmäßige Ausleihtermine, die aktuell nicht abbildbar sind (z.B. jeder erste Donnerstag im Monat)

Idee: sowohl feste Datumsangaben (kommasepariert) + relative Datumsangaben (http://docs.php.net/manual/en/datetime.formats.relative.php) ermöglichen, z.B.
second monday next month

@markus-mw markus-mw moved this from In progress to To do in commonsbooking (> v2.0.0) May 6, 2022
@markus-mw markus-mw moved this from To do to In progress in commonsbooking (> v2.0.0) May 6, 2022
@markus-mw markus-mw moved this from In progress to To do in commonsbooking (> v2.0.0) May 13, 2022
@hansmorb
Copy link
Contributor

Ich würde lieber ein npm package mit Feiertagen statt einer Webseite als API nehmen, ich halte das für zukunftssicherer.

@hansmorb hansmorb moved this from To do to In progress in commonsbooking (> v2.0.0) Oct 10, 2022
@hansmorb hansmorb assigned hansmorb and unassigned markus-mw Oct 10, 2022
@hansmorb hansmorb added this to the Heavy User Funktionen milestone Oct 22, 2022
@hansmorb hansmorb linked a pull request Dec 16, 2022 that will close this issue
@markus-mw markus-mw self-assigned this Feb 24, 2023
@markus-mw markus-mw linked a pull request Feb 24, 2023 that will close this issue
@markus-mw
Copy link
Contributor

markus-mw commented Feb 24, 2023

@chriwen , @hansmorb : Ich hab jetzt im experiment/timeframe-multiselect die Funktion für den Multiselect für Urlaub/Blockiert umgesetzt. Es wäre gut, wenn ihr das mal testen könntet. Sollte das soweit funktionieren, können wir noch einmal wegen der Urlaubs API Sache und der Kategorieeinstellung sprechen.

@chriwen
Copy link
Member

chriwen commented Feb 26, 2023

@markus-mw : Vielen Dank für die Umsetzung. Ergebnis erste Tests:

  • scheint grundsätzlich gut zu funktionieren
  • Habe testweise die Multi-Artikel-Auswahl für Buchungen eingeblendet (in der admin.js) und auch das scheint erstmal direkt zu funktionieren. Also: eine Station + mehrere Artikel. Das wäre natürlich super, wenn wir das in dem Feature direkt mit ausrollen können.

Was mir noch aufgefallen ist:

  • Wir bräuchten beim Typ Buchungen eine Validierung auf Überlappung der Artikel mit bereits bestehenden Timeframes. Das gibt es ja schon auf Einzel-Artikelebene, müsste nun auch irgendwie mit der Multiauswahl zurechtkommen.

@markus-mw
Copy link
Contributor

@chriwen, @hansmorb : Ich habe die Multi-Item Auswahl für Buchbare Timeframes inkl. der Übernahme einer eventuell vorhandenen Single-Item Auswahl (Abwärtskompatibilität) und der Validierung bzgl. bestehender Timeframes umgesetzt. Bitte teste das auch noch einmal.
Was aus meiner Sicht noch offen wäre:

  • die Konfiguration von Feiertagen/Geschlossen über eine Kategorie, wobei wir dazu noch einmal Videofonieren sollten
  • die Sache mit den Feiertagen per API blockieren, da würde ich aber ebenfalls gern noch einmal sprechen, wie genau das laufen soll
  • die "Alle" Option für Items/Locations sollten wir auch noch einmal besprechen, das ist nicht ganz so einfach wie es vielleicht aussieht, weil wir dabei ja die Nutzerberechtigungen einbeziehen müssen

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