commonsbooking icon indicating copy to clipboard operation
commonsbooking copied to clipboard

Behebe "Alle" Option verfügbar für CB Manager

Open hansmorb opened this issue 1 year ago • 3 comments

Danke @genevievecory . Ich habe hier die Logik etwas umgebastelt. Ursprünglich war es so, dass diese "Alle" Option quasi der "keine" Option entsprochen hat. Also wenn der Wert leer war, wurde davon ausgegangen, dass es auf Alle gesetzt ist. Jetzt muss ein spezifischer "Alle" Meta Key gesetzt werden der auch nur für Admins sichtbar ist.

Potentielle Probleme:

  • Alte TFs, die noch alle haben gehen weiterhin, auch wenn sie vielleicht nur von CB Managern erstellt wurden
  • Der neue "Alle" Meta Key ist keine Integer sondern ein String, das könnte vielleicht Probleme bereiten.

TODOS:

  • [x] - Wenn ein CB Manager eine Restriction mit dem "Please select" erstellt, wird diese auch noch wie eine "Alle" restriction behandelt (behoben)
  • [x] Merge an den Master

hansmorb avatar Jul 09 '23 14:07 hansmorb