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

Upgrade 0.27.3 #275

Closed
Naadjjaa opened this issue May 31, 2023 · 22 comments
Closed

Upgrade 0.27.3 #275

Naadjjaa opened this issue May 31, 2023 · 22 comments
Assignees

Comments

@Naadjjaa
Copy link
Collaborator

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

@bsantschi
Copy link
Collaborator

Offerte erstellt und an Nadja gesendet.

@Naadjjaa
Copy link
Collaborator Author

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

@Naadjjaa Naadjjaa changed the title Upgrade 0.27 Upgrade 0.27.3 Jun 23, 2023
@bsantschi bsantschi moved this from Backlog to Ready in Multitenant decidim.swiss Jul 3, 2023
@bsantschi bsantschi assigned carlobeltrame and unassigned Naadjjaa Jul 3, 2023
@carlobeltrame carlobeltrame moved this from Ready to In Progress in Multitenant decidim.swiss Jul 5, 2023
@Naadjjaa
Copy link
Collaborator Author

@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.

@Naadjjaa
Copy link
Collaborator Author

@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.

@carlobeltrame
Copy link
Collaborator

carlobeltrame commented Jul 14, 2023

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 👍

@bsantschi bsantschi assigned Kagemaru and Naadjjaa and unassigned carlobeltrame and Naadjjaa Jul 18, 2023
@Kagemaru
Copy link
Collaborator

Kagemaru commented Jul 20, 2023

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.

@Kagemaru Kagemaru pinned this issue Jul 20, 2023
@Kagemaru Kagemaru assigned innosmith and Naadjjaa and unassigned Kagemaru Jul 20, 2023
@bsantschi bsantschi moved this from In Progress to Testable on INT in Multitenant decidim.swiss Jul 21, 2023
@larsUE
Copy link
Contributor

larsUE commented Jul 21, 2023

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

@larsUE larsUE moved this from Testable on INT to Ready in Multitenant decidim.swiss Jul 21, 2023
@bsantschi bsantschi assigned Kagemaru and unassigned innosmith and Naadjjaa Jul 21, 2023
@Naadjjaa
Copy link
Collaborator Author

Naadjjaa commented Jul 25, 2023

@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

@innosmith
Copy link
Collaborator

innosmith commented Jul 25, 2023

@Kagemaru : die Anzeige von Veranstaltungen von Organisation führt zu einem Serverfehler:
https://test.deinklima.ch/assemblies/klimaneutraleregion/f/534/
https://test.deinklima.ch/processes/klima-idee-thun/f/751/

@innosmith
Copy link
Collaborator

innosmith commented Jul 25, 2023

@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.

@innosmith
Copy link
Collaborator

@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:

@Kagemaru
Copy link
Collaborator

Kagemaru commented Jul 27, 2023

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.

@bsantschi
Copy link
Collaborator

bsantschi commented Jul 28, 2023

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
Copy link
Contributor

larsUE commented Jul 28, 2023

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).

@Naadjjaa
Copy link
Collaborator Author

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

@carlobeltrame
Copy link
Collaborator

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.

@carlobeltrame
Copy link
Collaborator

@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
Copy link
Contributor

larsUE commented Aug 23, 2023

@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 :)

@Naadjjaa
Copy link
Collaborator Author

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

@innosmith
Copy link
Collaborator

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

@Naadjjaa
Copy link
Collaborator Author

Naadjjaa commented Sep 4, 2023

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

@bsantschi
Copy link
Collaborator

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

@Kagemaru Kagemaru moved this from Testable on INT to Deployed on PROD in Multitenant decidim.swiss Sep 11, 2023
@Naadjjaa Naadjjaa closed this as completed Oct 3, 2023
@github-project-automation github-project-automation bot moved this from Deployed on PROD to Done in Multitenant decidim.swiss Oct 3, 2023
@github-project-automation github-project-automation bot moved this to Done in OCL Aug 30, 2024
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

6 participants