Term-Customizer scoping für Prozessgruppen ermöglichen

[larsUE]

Wir brauchen das dringend. Gemäss Antti würden sie gerne einen PR zu diesem Thema akzeptieren: https://github.com/mainio/decidim-module-term_customizer/issues/75

[carlobeltrame]

Das verlinkte Issue spricht nur von der Landing Page von einer Prozessgruppe. @larsUE braucht ihr das auch nur auf dieser einen Seite oder wollt ihr es in allen Prozessen der Prozessgruppe haben?

[larsUE]

Genau, wir brauchen die Möglichkeit, zusätzlich auch die Landing Pages der Prozessgruppen auswählen zu können. Es müssen nicht alle zugewiesenen Prozesse ebenfalls mit ausgewählt werden, das würde die Flexibilität wohl etwas verringern und die Komplexität erhöhen. Decidim Awesome macht das bereits sehr schön, schau beispielsweise hier auf der Prod beim obersten Feld.

Die oberste Constraint setzt die Landing Page, die untere einen Prozess. Wäre cool, das könnten wir beim TC auch so machen.

[carlobeltrame]

Grobschätzung: 4 PT

Ich habe das mal angeschaut. Die Constraints von Decidim Awesome funktionieren leider anders als die beim Term Customizer, weil der Term Customizer sich "tiefer" / "interner" einnisten muss um seine Arbeit zu verrichten. Wir können daher nur punktuell von Decidim Awesome lernen. Ich habe unten die technischen Resultate meiner Analyse festgehalten. Im Detail ist es leider relativ kompliziert, was zu dem gefühlt eher hohen Aufwand für die aus User-Sicht wahrgenommene Featuregrösse führt. Der "Wurm" steckt darin dass Prozessgruppen (im Gegensatz zu Prozessen und Assemblies) keine Participatory Spaces sind, und wir die Filterung danach nicht einfach analog zu Prozessen "aktivieren" können, sondern quasi auf ganzer Länge der bestehenden Filterung Spezialfälle für Prozessgruppen einbauen müssen.

Tech spec

• [ ] In Menü [1] [2] eine neue Art von non-ParticipatorySpce Constraint ermöglichen, analog wie es Decidim Awesome macht
  • Allenfalls sind für die Speicherung noch Anpassungen im Formular bei der Validierung o.ä. nötig
• [ ] Im ControllerContext auch die ProcessGroup detektieren, falls wir gerade eine Prozessgruppen-Landing-Page rendern: in der Base Klasse und in der abgeleiteten Klasse die sich um den Kontext bei HTTP Requests kümmert
  • Diese Detektion, angenommen man hat einen Request, wird in Decidim Awesome in dieser Datei gemacht.
  • Wie man im ControllerContext überhaupt an den Request kommen kann ist noch unklar. Notfalls vielleicht ähnlich wie hier
• [ ] Die neue Information (auf welcher Prozessgruppen-Landingpage sind wir, wenn überhaupt) an den Resolver weitergeben: hier und hier
• [ ] Im Resolver eine Query-Erweiterung schreiben, die nur aktiv ist wenn wir die Landing Page einer Prozessgruppe rendern, und die Constraints auf diese spezifische Prozessgruppe (via id) oder auf alle Prozessgruppen (wenn id leer) sucht.
• [ ] Für alle oben erwähnten angefassten Dateien Tests schreiben
• [ ] Pull request beim Term Customizer eröffnen, auf Antwort warten und allfälligen Änderungswünschen von mainio nachkommen

[larsUE]

Ok, vielen Dank für die detaillierte Angaben! Ich bespreche das mit Werner, vorerst bitte nicht daran arbeiten.

[eliegaboriau]

Hello, We might be interested in this feature at Open Source Politics. Do you plan to work on this ? If you do, when do you think will it be available ? If you don't, we could work on it but it would be great to have some technical help if you already started to work on it. Thanks a lot

[larsUE]

Hey @eliegaboriau ! Cool, thanks for letting us know. We currently are not working on this, because we found another purpose for process groups. However, this still is interesting for us and we can supply some technical help if you wish. Just ask @carlobeltrame or @Kagemaru :) Thank you!

[larsUE]

English translation of the Tech Specs. @carlobeltrame please check if this makes sense:

Tech spec

• [ ] Enable new type of Non-ParticipatorySpaceConstraint in Menu [1] [2] e.g. like Decidim Awesome
• [ ] Potentially some adjustments to the form validations are required.
• [ ] Detect the Process Group in the ControllerContext, in case a process group landing page is being rendered: in the base class and in the derived class which builds the context for HTTP Requests
  • This detection, given that one has a request, is done in this file in Decidim Awesome.
  • How we can get the request in the ControllerContext is unclear. Maybe something similar as done in hitobito
• [ ] Transmit the new information (on which process group landingpage we are, if any) to the resolver: here and here
• [ ] Write a Query-Extension in the Resolver, which is only active when a landing page of a process group is rendered, and which searches if the constraint is valid for this specific process group (via ID) or for all process groups (no ID).
• [ ] Write tests for all aforementioned files
• [ ] Open PR at Term Customizer and adjust according to the wishes of mainio

[eliegaboriau]

Hello @larsUE and @carlobeltrame , i've submitted a solution to this issue. It is slightly less complicated than it was supposed, but it seems to work well. Feel free to add your review

[larsUE]

Implemented by @eliegaboriau in https://github.com/mainio/decidim-module-term_customizer/pull/94, thanks a lot!

We will need to update the term customizer as soon as the PR is approved.