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

Mitwirkungsprozesse aktualisieren nicht bei Filtereinstellungen #366

Closed larsUE closed 1 year ago

larsUE commented 1 year ago

Wenn ich einen Scope/Area-Filter setze bei der Übersicht auf der Seite /Processes, dann wird die Seite nicht automatisch aktualisiert. Mit einem Reload lande ich auf den gefilterten Einstellungen.

Kagemaru commented 1 year ago

Repariert & deployed auf INT

larsUE commented 1 year ago

@Kagemaru das funktioniert bei mir leider noch nicht. Test Case: hier das Thema "Verkehr" auswählen. Es sollte nur der VBZ Testprozess erscheinen.

Kagemaru commented 1 year ago

Ticket für Filter entfernen Bug: https://github.com/decidim/decidim/issues/8611 Redesign Ticket: https://github.com/decidim/decidim/issues/10148 Scope filter Bug Ticket: https://github.com/decidim/decidim/issues/10552

larsUE commented 1 year ago

@Kagemaru Kannst du abschätzen, was es bedeutet, dies zu flicken? Auf Meta.Decidim funktionieren die Filter (siehe: https://meta.decidim.org/processes) ausser, wie von Andres im Ticket https://github.com/decidim/decidim/issues/10552 erwähnt, in den Prozessgruppen. Könntest du eine Aufwandschätzung machen für a) Flicken bei /processes (nur Zürich) b) Flicken bei Process groups (inkl. PR bei Decidim, aufbauend auf release/0.27-stable

larsUE commented 1 year ago

Noch zur Info: In Luzern funktioniert dies wie gewünscht: https://dialogluzern.ch/processes?filter%5Bscope_id%5D%5B%5D=21&filter%5Barea_id%5D=

Kagemaru commented 1 year ago

Dies scheint ein Problem mit der modifizierten Gruppenanzeige zu sein. Ich schätze 1–2 Tage Arbeit.

larsUE commented 1 year ago

Nach Absprache mit Werner kann das umgesetzt werden.

larsUE commented 1 year ago

Bitte Zusammenhang mit https://github.com/decidim/decidim/pull/10829 prüfen - allenfalls entsteht da bereits ein Fix.

larsUE commented 1 year ago

Und same mit diesem hier: https://github.com/decidim/decidim/pull/11248

larsUE commented 1 year ago

Die oben erwähnten PRs sind sind teil von Decidim V0.27.4. Bitte also zuerst https://github.com/puzzle/decidim-zuerich/projects/4#card-90016669 ausführen, und danach dieses Ticket nochmals anschauen :)

larsUE commented 1 year ago

Ebenfalls potenziell relevant: https://github.com/decidim/decidim/pull/11438 @Kagemaru kannst du dich mal mit diesem Ticket befassen?

carlobeltrame commented 1 year ago

Es sind dieselben Symptome wie es bei #196 schon waren. Der Browser requestet Accept: text/javascript, application/javascript, application/ecmascript, application/x-ecmascript, */*; q=0.01, aber der Server antwortet (nur bei Zürich) fälschlicherweise mit dem HTML-Inhalt. Somit fehlen die Instruktionen zum aktualisieren der Seite, und die gefilterten Prozesse tauchen nie auf.

In #196 hat @Kagemaru das schlussendlich gelöst, indem er ein altes Override entfernt hat. Ich bin mir jetzt nicht sicher, wie er zum Schluss kam dass das die Lösung sei, und auch nicht, was die analoge Lösung in diesem Fall hier wäre.

Kagemaru commented 1 year ago

Ich bin nun etwas weiter. Wenn ich das Template index.html.erb entferne geht es.

Mit dem Template kommt das ganze Layout zurück.

Wenn ich aber gleichzeitig noch index.js.erb überschreibe mit dem gleichen Content wie in decidim, funktioniert es wieder normal.

Aber anstatt dies weiter zu verfolgen, würde ich aus dem Template override gerne ein Deface override machen. Dann sollten wir dieses Problem in Zukunft nicht mehr haben.

Kagemaru commented 1 year ago

Mit deface funktioniert jetzt die .js Antwort wieder. Aber leider verhält sich der Filter noch nicht richtig.

Da ich morgen nicht für Puzzle arbeite, kannst du, @carlobeltrame morgen mal reinschauen, ob du es zum Laufen bringen kannst?

carlobeltrame commented 1 year ago

Ich konnte es jetzt fixen. Wenn man einen Filter auswählt und den Dialog schliesst werden jetzt die korrekten Teile der Seite neu geladen. Wenn man danach auf die Filter Pille klickt, verschwindet diese und es hat keine Wirkung. Das ist aber auch auf DialogLuzern und sogar auf MetaDecidim so. Darum habe ich das vorerst noch nicht weiter verfolgt. Fix ist jetzt auf INT aufgeschaltet.

larsUE commented 1 year ago

Ich habe das ebenfalls getestet und komme zum gleichen Schluss wie du @carlobeltrame . Alles andere funktioniert. Da im Redesign eh ganz viele Bug fixes kommen und die Filter auch anders aussehen, bitte keine Zeit mehr aufwenden.