puzzle / decidim-zuerich

Mitwirkungsportal für die Stadt Zürich, basierend auf Decidim
https://mitwirken.stadt-zuerich.ch
GNU Affero General Public License v3.0
8 stars 2 forks source link

Prozessschritt nur an einem Tag #221

Closed larsUE closed 2 years ago

larsUE commented 2 years ago

Prozessschritte müssen derzeit eine Zeitspanne von mehreren Tagen abdecken. Viele unserer Prozesse haben jedoch einzelne Veranstaltungen, die wichtig sind und als eigenständige Prozessschritte zu verstehen sind.

Deshalb folgendes anpassen:

Kagemaru commented 2 years ago

Ich hab mal kurz die Validations angeschaut. Ich denke, dieser Ansatz sollte funktionieren:

start_options = Decidim::ParticipatoryProcessStep._validators[:start_date].first.options
end_options = Decidim::ParticipatoryProcessStep._validators[:end_date].first.options

new_start_options = start_date.options.except(:before).merge(before_or_equal_to: :end_date)
new_end_options = start_date.options.except(:after).merge(after_or_equal_to: :start_date)

Decidim::ParticipatoryProcessStep._validators[:start_date].first.instance_variable_set(:@options, new_start_options)
Decidim::ParticipatoryProcessStep._validators[:end_date].first.instance_variable_set(:@options, new_end_options)

Ich probiere dies sobald ich mehr Zeit habe.

larsUE commented 2 years ago

Carlo hat richtig vermutet, dass nur ein Startdatum (oder gar kein Datum) ausgefüllt werden muss. Nicht ausgefüllte Daten werden mit einem Fragezeichen angezeigt. Wir können also die Validators sein lassen (bzw. das ist sogar gut, dass sie so funktionieren). Wir bräuchten also eine Anpassung, die nur einzelne Daten anzeigt, falls vorhanden, und nicht die Spanne mit Fragezeichen.

Hier ein paar Cases:

Bildschirmfoto 2022-01-18 um 12 35 23
Kagemaru commented 2 years ago

Ich habe mal diese Formate verwendet: decidim_zuerich_step_dates2 decidim_zuerich_step_dates1

Ich werde es mal so auf INT deployen. Melde dich falls du ein anderes Format wünschst.

larsUE commented 2 years ago

Das sieht super aus! Ich sehe es noch nicht auf der INT & nehme an, du bist noch dran. Funktioniert diese Darstellung auch im Schrittübersichtsfenster?

Bildschirmfoto 2022-01-19 um 16 25 20
Kagemaru commented 2 years ago

@larsUE Ja, hatte es noch nicht getagt. Nun ist es drauf.

larsUE commented 2 years ago

Funktioniert prima, vielen Dank! Könntest du Ende Woche einen Prod-Release machen, denn die Änderung wird zu Beginn nächster Woche gebraucht.