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

Mind. 15 Buchstaben wegkriegen #270

Closed
Naadjjaa opened this issue May 9, 2023 · 18 comments
Closed

Mind. 15 Buchstaben wegkriegen #270

Naadjjaa opened this issue May 9, 2023 · 18 comments
Assignees

Comments

@Naadjjaa
Copy link
Collaborator

Naadjjaa commented May 9, 2023

Damit das Update der Core & Awesome Version an Dringlichkeit verliert, brauche ich bis am 22.05. zwingend einen übergangsmässigen Quickfix für dass die Eingabefelder der "Reporting Proposals"-Komponente Titel und Kurzbeschrieb nicht mind. 15 Buchstaben enthalten müssen und keine Pflichtfelder sind. (Bereits in Mail vom 20.04. erwähnt).

image

@larsUE
Copy link
Contributor

larsUE commented May 11, 2023

Winterthur wäre an diesem Fix auch fürs allgemeine Vorschlagsmodul interessiert.

@carlobeltrame
Copy link
Collaborator

Wir können versuchen, die Änderungen von Decidim Awesome für den Moment zu uns zu übernehmen. Wenn wir das für alle Tenants und gleichermassen für Vorschläge und Schadensmeldungen umsetzen dürfen, vereinfacht das die Sache.

@larsUE
Copy link
Contributor

larsUE commented May 11, 2023

Von Seiten Cokreation und gemeinsamer.space wäre das super. @innosmith ich nehme an, von Seiten deinklima und köniz gibts auch keine Vetos?

Bezüglich Update besprechen wir dann ja nächsten Dienstag mehr :) Ich könnte mir auch vorstellen, direkt auf Version 27 zu wechseln, statt auf 26.4

@carlobeltrame
Copy link
Collaborator

Ein Update wäre natürlich die beste Option, aber dann sind wir nicht zu 100% sicher dass bis am 22.5. alles wirklich rund läuft. Darum gibt es eben dieses Issue hier.

@Naadjjaa
Copy link
Collaborator Author

@carlobeltrame Das wäre super!
Es ist ein Detail aber scheint uns für die Einführung des Schadenmelders sehr relevant. Wir haben schon durch die nötige Registrierung eine relativ grosse Hürde und möchten nicht, dass Meldungen nicht abgeschickt werden, weil Fehlermeldungen aufgrund von zu wenigen Ziffern die Leute vom Posten abhalten.
@innosmith Danke für eine zeitnahe Rückmeldung ob wir das so machen dürfen.

@innosmith
Copy link
Collaborator

Der vorgeschlagene Quickfix passt für mich @Naadjjaa . Die Thematik v0.27 kann ich mit meinem aktuellen Kenntnisstand nicht beurteilen.

@Naadjjaa
Copy link
Collaborator Author

@carlobeltrame Wir haben grünes Licht für den Quickfix für alle decidim.swiss Tenants. Danke fürs zeitnahe umsetzen.

@carlobeltrame
Copy link
Collaborator

Ich habe das umgesetzt und auf Integration ausgerollt. Zur Info: Ich habe es gleich wie Decidim Awesome umgesetzt, und somit gilt der Fix nur im Proposal Create Wizard im Frontend. Wenn man Vorschläge direkt im Admin-Backend erstellt, werden die Regeln wie bisher durchgesetzt.

@Naadjjaa @innosmith @larsUE bitte seid euch bewusst dass es eine schlechte Idee sein kann, Vorschläge ohne Titel und ohne Textkörper zuzulassen. Auf Tenants die die Vorschlags-Kacheln angepasst haben und den "Vorschlag anzeigen" Button versteckt haben, ist es schwierig, einen titellosen Vorschlag zu öffnen.
Screenshot 2023-05-15 at 16-59-37 Vorschläge - Echogruppe Dialog Luzern - Dialog Luzern
Bei "letzte Aktivität" gibt es ebenfalls keinen Link zum Vorschlag, wenn kein Titel vorhanden ist.
Screenshot 2023-05-15 at 17-04-43 Dialog Luzern
Ich bin gespannt von euren Erfahrungen mit der Qualität der Vorschläge / Ideen / Meldungen zu hören, jetzt wo diese Qualitäts-Safeguards ausgeschaltet sind.

@Naadjjaa
Copy link
Collaborator Author

@carlobeltrame Danke für deinen Hinweis! Sobald wir die aktuellste Version von Awesome haben müssten wir diesen Quickfix wieder entfernen. Dann wird die Anzahl Buchstaben konfigurierbar sein - wir werden diese dann beim Titel wieder auf 3 Zeichen setzen.

@Naadjjaa
Copy link
Collaborator Author

@carlobeltrame Funktioniert bei meinem Test auf INT nicht:
image

@carlobeltrame
Copy link
Collaborator

@Naadjjaa hier funktionierts aber: https://test.dialogluzern.ch/assemblies/reussbuehl/f/57/proposals/new

Kannst du mir den genauen Link schicken wo du getestet hast? Oder hast vielleicht versehentlich auf PROD getestet?

@Naadjjaa
Copy link
Collaborator Author

@carlobeltrame Kann es sein, dass du das nur für Proposals angepasst hast, aber nicht für Reporting Proposals?
Hier funktionierts noch nicht:
https://test.dialogluzern.ch/processes/nadjaprozess/f/764/proposals/new

@Naadjjaa Naadjjaa assigned carlobeltrame and unassigned Naadjjaa May 23, 2023
@carlobeltrame
Copy link
Collaborator

Danke für den Link. Das wird es wohl sein. Ich habe es genau so umgesetzt wie es in Decidim Awesome auch gemacht wurde, heisst es hätte also mit Decidim Awesome auch nicht funktioniert.

Die Reporting Proposals basieren stark auf den normalen Proposals, und ich habe auf den ersten Blick nicht gesehen, wo Pokecode dieses Formular für die Reporting Proposals separat abgeändert hat (sie haben da ausnahmsweise aus mir noch unbekannten Gründen nicht den direktesten Weg gewählt).

Somit muss ich diesen Fix noch einmal neu für die Reporting Proposals implementieren, durchtesten und auf INT ausrollen.

@Naadjjaa
Copy link
Collaborator Author

@carlobeltrame Für die Reporting Proposals kann man neu die Anzahl Buchstaben direkt im Awesome Modul konfigurieren. Es hat auf unserer Entwicklungsumgebung funktioniert. Lass uns um 9 kurz besprechen, wie dringend Matthias dieses Feature sieht.

@Naadjjaa
Copy link
Collaborator Author

image

@carlobeltrame
Copy link
Collaborator

Interessant. Bist du sicher dass das auch für reporting proposals funktioniert, nicht nur für proposals? Ich sehe weder im Code von Decidim Awesome Referenzen zu Reporting Proposals, noch umgekehrt. Vielleicht habe ich aber bei Decidim Awesome etwas übersehen und sie machen es cleverer als ich auf die Schnelle erkennen konnte.

Klar, das GUI habe ich nicht von Decidim Awesome übernommen. Aber die Logik dahinter, die die eigentliche Einschränkung auf 15 Zeichen abändert, habe ich 1:1 übernommen so wie ich es in Decidim Awesome sehe.

@bsantschi
Copy link
Collaborator

Voraussichtlich mit Decidim v0.27 gelöst, daher on hold.

@Naadjjaa
Copy link
Collaborator Author

Naadjjaa commented Oct 3, 2023

Mit Fix auf Awesome nun gelösst.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

5 participants