stadtluzern/decidim-ocl

Upgrade 0.27.3

Closed this issue · 22 comments

Ich brauche bitte eine Offerte für das Upgrade.
Beachtet dabei bitte, ass die Kosten für Upgrades auch durch die anderen Städte mitgetragen werden (gemäss Detailauflistung Kostenzusammenstellung decidim.swiss).
Merci

Offerte erstellt und an Nadja gesendet.

@bsantschi Ivan meint, wir sollten unbedingt die neuste Version 0.27.3 verwenden, da diese wichtige Security fixes enthält.

@carlobeltrame "I'm happy to inform you that we made the official instructions for the reporting proposals. You can check it here: https://github.com/stadtluzern/decidim-letterbox-damagereporting. Also, will help developers which want to install the module".
Bez. PDF sollte bis morgen Mittag die Version bereitstehen. Ich melde mich dann im anderen Ticket diesbezüglich nochmals.

@carlobeltrame Danke für das Update. Macht ihr Decidim Awesome auch noch auf die 0.27 Version? Wir warten seit dem Launch des Schadenmelders auf eine Funktion, die darin enthalten ist.

Fehler in decidim.swiss-spezifischen Code-Anpassungen, die noch behoben werden müssen bevor 0.27 funktionieren kann:

@Naadjjaa mit dem Upgrade kommt dann gleich die aktuellste Version von Decidim Awesome mit 👍

Das Upgrade ist jetzt live auf INT. Ich habe mal kurz durchgeklickt und es sieht gut aus (sogar Notifications 😲).

Bei weiteren Fehlern, bitte hier melden, das Ticket nach Ready verschieben und mich assignen.
Dann behalte ich das Ticket im Auge.

larsUE commented

Der File-Upload wirft derzeit noch einen Fehler -> egal was ich hochlade kommt immer "Fehler!" (hatten wir in Zürich ja auch).

@Kagemaru Das laden der Assemblies-Seite im Backend hat schon vorher lange gedauert, jetzt erhalte ich jedoch einen Serverfehler (Firefox & Chrome):

Image
https://test.dialogluzern.ch/admin/assemblies

@Kagemaru : im Backend unter Benutzer gibt es neu die Menüoption "Admin Verantwortlichkeit". Ein Klick darauf von der deinklima-Testplattform listet Admins oder User aller Decidim-Tenants auf, siehe:
https://test.deinklima.ch/admin/decidim_awesome/admin_accountability

Insgesamt sind dies 37 Seiten von Admins oder Users inkl. Emailadressen. Das darf doch nicht sein? Ist das ein Feature speziell für Luzern entwickelt? Ich hoffe ihr könnt dies rasch ändern, bevor unsere Kund:innen aufschrecken bez. Datenschutz!

Edit nvb: wurde bereits an Pokecode gemeldet.

@Kagemaru : das hinzufügen eines Profilbildes ergibt bei mir jeweils einen Fehler. Am kleinen Bild selbst sollte es nicht liegen, denn dieses habe ich schon öfter dafür verwendet.

Hier der direkte Link, getestet auf ImpulsKöniz und deinklima:

Ich fasse mal zusammen:

  • Bilder Upload ⇨ CORS Regeln angepasst
  • Backend Assemblies ⇨ INT Timeout auf 120s erhöht. Request dauert 72s 😲
  • Meetings von Organisationen ⇨ default filter aktualisiert
  • Admin Verantwortlichkeiten ⇨ Was soll ich da machen? 😕
  • Meetings Sortierung ⇨ Sortierung angepasst

Diese Changes sind nun auf INT deployed.

Danke viu mau, Thomas! Betreffend "Admin Verantwortlichkeiten" warten wir auf einen Fix von Pokecode. Habe dazu ein eigenes Ticket erstellt: decidim-swiss/decidim.swiss#96

larsUE commented

Datei Upload funktioniert wieder. Was mir noch auffällt ist, dass es nach wie vor missing translations hat, obwohl ich das Sprachfile schon länger auf dem aktuellen Stand habe (allerdings erst nachdem 0.27.3 erschienen ist). Vielleicht behebt sich das mit 0.27.4).

@carlobeltrame Ich finde "Navigation Maps" nicht mehr auf INT (bei den Blöcken für die Startseite). Ging dies vergessen?

Navigation Maps wurde noch nicht für 0.27 geupdatet, ein entsprechender Vorschlag ist seit Februar offen: Platoniq/decidim-module-navigation_maps#32

Ich kann probieren, das Modul mit den vorgeschlagenen Änderungen zu installieren.

Ebenfalls noch nicht kompatibel sind die Module für Navbar Links und URL Aliases.

@Naadjjaa @larsUE @innosmith die Integrationsumgebung von decidim.swiss ist jetzt auf Decidim 0.27.4.
Zusätzlich sind noch die aktuellsten Versionen der pokecode-Module drauf, und eine Version von navigation maps die mit 0.27 kompatibel sein müsste.
Als nächstes kümmere ich mich noch um die Karten-Integration.

larsUE commented

@carlobeltrame Danke! Ich denke Navbar Links und URL Aliases werden auch nicht mehr gebraucht, da beide Funktionen mittlerweile in Decidim Awesome integriert wurden. Das müssen aber @innosmith und @Naadjjaa entscheiden :)

@carlobeltrame @larsUE @innosmith Wir brauchen auch schon lange nur noch die Funktionen von Awesome für URL & Links.

@carlobeltrame @Naadjjaa @larsUE : wir setzen auch nur noch auf Decidim Awesome.

#275 (comment) Ich komme mittlerweile gar nicht mehr auf die Assemblies Seite im Backend, aufgrund des Timeouts. @bsantschi

Das Timeout mit den Assemblies wird in einem eigenen Ticket weiterverfolgt: decidim-swiss/decidim.swiss#104