Vereinsmeldung: nach „Wieder öffnen“ nur noch Ansicht, Abschliessen nicht mehr möglich #158

New issue

Open

opened 2026-10-28 18:34:10 +00:00 by claude · 1 comment

claude commented

2026-10-28 18:34:10 +00:00

Collaborator

Beobachtet (Olli, 28.10.2026, lokal gegen die Dev-DB = Staging-Stand)

Als Admin im Meldungseingang eine abgeschlossene Vereinsmeldung geöffnet (baloise) und oben auf „Wieder öffnen“ geklickt. Danach:

oben steht „Nur Ansicht“, es gibt weder „Speichern“ noch „Abschliessen“;
die Meldung lässt sich auf dieser Seite nicht mehr abschliessen;
auf dem Server steht sie jetzt auf „in Arbeit“.

Wahrscheinliche Ursache (aus dem Code, noch nicht nachgestellt)

„Nur Ansicht“ erscheint für den Admin nur, wenn die Seite mit ?mode=view geöffnet ist: viewOnly = viewOnlyParam || (!isAdmin && …) (portal/src/pages/ClubPage.tsx:55). Mit ?mode=view öffnen die grünen Schritte „Meldung eingegangen“ im Meldungseingang (RegistrationListPage.tsx:186, :196); „Bearbeiten“ öffnet ohne (:372).
„Wieder öffnen“ wird auch in der Nur-Ansicht angeboten: ClubPageHeader.tsx zeigt es bei club.completed && isAdmin, ohne viewOnly zu prüfen. „Speichern“ und „Abschliessen“ dagegen nur ohne viewOnly.
Folge: In der Nur-Ansicht kann der Admin die Meldung öffnen, aber nicht mehr abschliessen. Die Seite bleibt eine Sackgasse, bis man sie über „Bearbeiten“ neu öffnet.

Falls es auch über „Bearbeiten“ passiert, stimmt diese Erklärung nicht. Das ist beim Beheben zuerst nachzustellen.

Zweiter Grund, warum Abschliessen hier scheitern kann

Bei baloise fehlt im Reiter „Vereinsadressen“ der grüne Haken: Beim Präsidenten ist Hausnummer leer (Pflichtfeld), die Nummer steht im Feld Strasse („Bergweg 6“). Auch über „Bearbeiten“ lässt sich die Meldung darum erst nach dem Nachtragen abschliessen. Wieder geöffnete Altmeldungen mit solchen Daten brauchen dafür einen Hinweis, oder die Prüfung beim Abschliessen durch den Admin wird anders gefasst. Das ist beim Beheben zu entscheiden.

Nach dem Meldeschluss

Die Saison steht auf „läuft“. Der Spielleiter kann eine wieder geöffnete Meldung nach dem Meldeschluss nicht selbst abschliessen (viewOnly für Nicht-Admins bei registrationClosed). Wer öffnet, muss also selbst wieder abschliessen können.

Erwartet

„Wieder öffnen“ führt auf eine Seite, auf der man bearbeiten und abschliessen kann. Entweder gibt es den Knopf nur ohne Nur-Ansicht, oder er hebt die Nur-Ansicht auf.
Eine wieder geöffnete Meldung lässt sich vom Admin wieder abschliessen.

## Beobachtet (Olli, 28.10.2026, lokal gegen die Dev-DB = Staging-Stand) Als Admin im Meldungseingang eine abgeschlossene Vereinsmeldung geöffnet (baloise) und oben auf **„Wieder öffnen“** geklickt. Danach: - oben steht **„Nur Ansicht“**, es gibt weder „Speichern“ noch „Abschliessen“; - die Meldung lässt sich auf dieser Seite nicht mehr abschliessen; - auf dem Server steht sie jetzt auf „in Arbeit“. ## Wahrscheinliche Ursache (aus dem Code, noch nicht nachgestellt) - „Nur Ansicht“ erscheint für den Admin nur, wenn die Seite mit `?mode=view` geöffnet ist: `viewOnly = viewOnlyParam || (!isAdmin && …)` (`portal/src/pages/ClubPage.tsx:55`). Mit `?mode=view` öffnen die grünen Schritte „Meldung eingegangen“ im Meldungseingang (`RegistrationListPage.tsx:186`, `:196`); „Bearbeiten“ öffnet ohne (`:372`). - **„Wieder öffnen“ wird auch in der Nur-Ansicht angeboten**: `ClubPageHeader.tsx` zeigt es bei `club.completed && isAdmin`, ohne `viewOnly` zu prüfen. „Speichern“ und „Abschliessen“ dagegen nur ohne `viewOnly`. - Folge: In der Nur-Ansicht kann der Admin die Meldung öffnen, aber nicht mehr abschliessen. Die Seite bleibt eine Sackgasse, bis man sie über „Bearbeiten“ neu öffnet. Falls es auch über „Bearbeiten“ passiert, stimmt diese Erklärung nicht. Das ist beim Beheben zuerst nachzustellen. ## Zweiter Grund, warum Abschliessen hier scheitern kann Bei baloise fehlt im Reiter „Vereinsadressen“ der grüne Haken: Beim Präsidenten ist **Hausnummer** leer (Pflichtfeld), die Nummer steht im Feld Strasse („Bergweg 6“). Auch über „Bearbeiten“ lässt sich die Meldung darum erst nach dem Nachtragen abschliessen. Wieder geöffnete Altmeldungen mit solchen Daten brauchen dafür einen Hinweis, oder die Prüfung beim Abschliessen durch den Admin wird anders gefasst. Das ist beim Beheben zu entscheiden. ## Nach dem Meldeschluss Die Saison steht auf „läuft“. Der Spielleiter kann eine wieder geöffnete Meldung nach dem Meldeschluss nicht selbst abschliessen (`viewOnly` für Nicht-Admins bei `registrationClosed`). Wer öffnet, muss also selbst wieder abschliessen können. ## Erwartet - „Wieder öffnen“ führt auf eine Seite, auf der man bearbeiten und abschliessen kann. Entweder gibt es den Knopf nur ohne Nur-Ansicht, oder er hebt die Nur-Ansicht auf. - Eine wieder geöffnete Meldung lässt sich vom Admin wieder abschliessen.

claude added the

Kind

Bug

Priority

labels

2026-10-28 18:34:10 +00:00

claude commented

2026-10-28 18:38:02 +00:00

Author

Collaborator

Ursache bestätigt, Entscheid (Olli, 28.10.2026)

Nachgestellt: Im Meldungseingang öffnen die Schritte „In Bearbeitung“ und „Meldung eingegangen“ die Meldung mit ?mode=view (RegistrationListPage.tsx:186, :196), „Bearbeiten“ ohne (:372). Mit ?mode=view zeigt die Seite „Nur Ansicht“, unabhängig von der Rolle (ClubPage.tsx:55). „Wieder öffnen“ erscheint dort trotzdem (ClubPageHeader.tsx), Speichern und Abschliessen nicht. Über „Bearbeiten“ lässt sich die Meldung bearbeiten und abschliessen.

Der Ansehen-Modus kam am 03.04.2026 mit 1e2d744c („add read-only view mode for registrations“), ohne Begründung und ohne Spec.

Entscheid: Der Ansehen-Modus fällt weg. Ein Klick auf einen Schritt öffnet dieselbe Seite wie „Bearbeiten“. Der Verband darf ohnehin bearbeiten, gespeichert wird erst mit „Speichern“. Damit gibt es nur noch einen Eingang, und die Sackgasse nach „Wieder öffnen“ fällt weg.

Für den Bau:

?mode=view und viewOnlyParam raus (ClubPage.tsx, die zwei Links in RegistrationListPage.tsx). Sonst benutzt das niemand, auch kein E2E.
viewOnly bleibt für Nicht-Admins (abgeschlossen bzw. nach dem Meldeschluss).
portal/src/test/ClubPageHeader.test.tsx prüft „Nur Ansicht“ und muss angepasst werden (Teständerung, mit Olli abstimmen).
Offen bleibt der zweite Grund aus der Beschreibung: Pflichtfelder in Altdaten (Hausnummer leer, Nummer im Feld Strasse) verhindern das Abschliessen auch über „Bearbeiten“.

Auf der Dev-DB stehen nach dem Nachstellen baloise und Aegerter & Bosshardt AG auf „In Bearbeitung“.

## Ursache bestätigt, Entscheid (Olli, 28.10.2026) Nachgestellt: Im Meldungseingang öffnen die Schritte **„In Bearbeitung“** und **„Meldung eingegangen“** die Meldung mit `?mode=view` (`RegistrationListPage.tsx:186`, `:196`), **„Bearbeiten“** ohne (`:372`). Mit `?mode=view` zeigt die Seite „Nur Ansicht“, unabhängig von der Rolle (`ClubPage.tsx:55`). „Wieder öffnen“ erscheint dort trotzdem (`ClubPageHeader.tsx`), Speichern und Abschliessen nicht. Über „Bearbeiten“ lässt sich die Meldung bearbeiten und abschliessen. Der Ansehen-Modus kam am 03.04.2026 mit `1e2d744c` („add read-only view mode for registrations“), ohne Begründung und ohne Spec. **Entscheid: Der Ansehen-Modus fällt weg.** Ein Klick auf einen Schritt öffnet dieselbe Seite wie „Bearbeiten“. Der Verband darf ohnehin bearbeiten, gespeichert wird erst mit „Speichern“. Damit gibt es nur noch einen Eingang, und die Sackgasse nach „Wieder öffnen“ fällt weg. Für den Bau: - `?mode=view` und `viewOnlyParam` raus (`ClubPage.tsx`, die zwei Links in `RegistrationListPage.tsx`). Sonst benutzt das niemand, auch kein E2E. - `viewOnly` bleibt für Nicht-Admins (abgeschlossen bzw. nach dem Meldeschluss). - `portal/src/test/ClubPageHeader.test.tsx` prüft „Nur Ansicht“ und muss angepasst werden (Teständerung, mit Olli abstimmen). - Offen bleibt der zweite Grund aus der Beschreibung: Pflichtfelder in Altdaten (Hausnummer leer, Nummer im Feld Strasse) verhindern das Abschliessen auch über „Bearbeiten“. Auf der Dev-DB stehen nach dem Nachstellen **baloise** und **Aegerter & Bosshardt AG** auf „In Bearbeitung“.