Feedback Gewünscht: Proof-of-Conecpt

wp99cp commented 1 year ago

Ich habe heute den Prove-of-Concept etwas weiter getrieben und bin gespannt auf euer Feedback. @CeviSchweiz / @btemperli geht das in etwa in die Richtung, die ihr euch vorgestellt habt?

Link zum Proof-of-Concept: https://kurse.cevi.tools/

Hinweise / Discalmer

[ ] Die Daten werden aus der Hitobito Test-Instance (https://cevi.puzzle.ch/) geladen, daher stimmen sie nicht mit der Cevi-DB überein.
[ ] Die Daten werden einige Zeit gecached, bevor sie via Events API neu abgefragt werden. Im produktiven Betrieb kann die Cache-Dauer beliebig eingestellt werden.
[x] @nchiapol können wir das ganze mal mit einem Dachverband/RV Token testen? Gerne kannst du es auch lokal machen, einfach ein .env erstellen (kopieren aus example.env) und dann mit docker-compose up --build starten.
[x] Filterung via URL Parameter ist zur Zeit noch nicht möglich. #4
[ ] Ebenso eine Filterung nach Datum/Ort/Zielpublikum

nchiapol commented 1 year ago

@wp99cp Vielen Dank für deine Arbeit sieht cool aus. Wir können für die Entwicklung unsere Integrationsumgebung nutzen: https://cevi.puzzle.ch, bzw. https://cevi.puzzle.ch/groups/1/events/course?token=... Die hat sinnvollere Einträge und wird nicht reset.

(Token bekommst du gleich per Mail.)

wp99cp commented 1 year ago

@wp99cp Vielen Dank für deine Arbeit sieht cool aus. Wir können für die Entwicklung unsere Integrationsumgebung nutzen: https://cevi.puzzle.ch, bzw. https://cevi.puzzle.ch/groups/1/events/course?token=... Die hat sinnvollere Einträge und wird nicht reset.

(Token bekommst du gleich per Mail.)

Vielen Dank, ich habe das Config File entsprechend angepasst.

avrosch commented 1 year ago

Merci für die Arbeit - ein grosser Schritt vorwärts!

[ ] Ich wünschte mir für eine Kurs-/Event-Übersicht noch das Datum (z.B. Startdatum) in der "obersten" Listen-Ansicht. Ist das möglich?

btemperli commented 1 year ago

Danke @wp99cp für den PoC! Sehr cool & sieht schon mal toll aus 😃

Filter via URL & Filter nach Datum und so hast du ja bereits erwähnt, das ist sicher praktisch. Hier ein paar weitere Gedanken von mir:

[x] Bei Einbindung als iframe Logo und "unnötige" Layout-Elemente entfernen (via URL-Parameter, Beispiel: https://logo.cevi.ch/?iframe=true)
[ ] Gemäss @avrosch ein passendes Datum in die Haupt-Liste
[ ] Allenfalls Spalten via URL-Parameter ein- oder ausschalten?
[ ] Das Feld "Durchgeführt von" aus der DB übernehmen (also die jeweilige Gruppe / Region)
[ ] Anzeigen, wenn Kurs bereits ausgebucht ist oder das Datum in der Vergangenheit liegt (oder die Kurse in diesem Fall gar nicht mehr anzeigen?) -> hide old events: https://github.com/cevi/event-overview-cevidb/commit/a2006fe69d549f67d3e9b7b6a6f0c8470b173337
[x] Anmelde-Link für Externe gleich übernehmen (falls möglich, habe den Kurs hier entsprechend angepasst: https://cevi.puzzle.ch/groups/300/events/1179)

@tschuepbach hast du weitere Ergänzungen?

Dann noch ein paar Details, die aber für den PoC nicht relevant sind:

[ ] Bei den Teilnehmerzahlen entweder 1 von 3 oder ausgebucht (statt 1 von 1) oder 15 (wenn keine maximale TN-Zahl definiert)
[ ] Berücksichtigen, dass die Beschreibung von Kursen ziemlich lange sein kann (siehe https://db.cevi.ch/groups/2/events/2885.html)

tschuepbach commented 1 year ago

Hallo zusammen

Ergänzungen meinerseits:

[ ] Zukünftig arbeiten wir mit Kurskategorien und Kursarten. Allenfalls in Filter berücksichtigen #7
[ ] Generell bin ich der Ansicht, dass die Filterung nach Kursarten am zentralsten ist. Die Nutzer*innen einer solchen Kursübersicht wissen meistens, ob sie ein J+S-Modul Lagerleitung oder ein J+S-MFE besuchen möchten. Allenfalls lässt sich die Filterung nach Kursarten/ -kategorien noch präsenter einbauen/platzieren.
[x] Motto scheint mir in der Kursübersicht als überflüssig resp. nicht wichtig -> fixed with https://github.com/cevi/event-overview-cevidb/commit/70e7851b25fe6856556f2507b3c39eebb68b9eb9
[x] Es wäre praktisch, wenn sich die einzelne Kursübersicht nicht nur über das "+" öffnen lassen würde, sondern sich die Schaltfläche über die ganze Zeile erstreckt #5
[ ] Mit den neuen DB-Update wird regionsübergreifende Sichtbarkeit von Events ermöglicht. Dies bezieht sich auf die Checkbox "Sichtbarkeit: Anlass ist für die ganze Datenbank sichtbar". Ich bin der Meinung, dass nur Events mit Sichtbarkeit = true auf einer solchen Kursübersicht erscheinen sollte. (verbandspolitisches Thema)
[x] Mobile Ansicht ist aus meiner Sicht aktuell noch nicht benutzerfreundlich resp. überladen (Wie kann mit langen Beschreibungen umgegangen werden?) #5

patrickuhlmann commented 10 months ago

Feedback von Galina - 30.12

Ich finde es noch unbefriedigend, dass ein Anlass nur zu einer Organisation gehören kann. Unsere Anlässe sind ja Anlässe des Cevi Schweiz (durchgeführt von der Fachgruppe International). Evt. würde es reichen, wenn man eine Ebene obendran die Organisation abfragt…
Irgendwie bräuchte es noch mehr inhaltliche Tags zu den Anlässen/Kursen, die man Filtern kann…

cevi / event-overview-cevidb

Feedback Gewünscht: Proof-of-Conecpt #3

Hinweise / Discalmer