search

hitobito / hitobito_youth

A hitobito wagon adding common features for (swiss) youth organizations.
Other
10 stars 20 forks source link

Change settings add source to before initialize #89

Closed njaeggi closed 2 weeks ago

njaeggi commented 2 weeks ago

For some reason further wagon settings.yml changes, have overwritten the Core Settings but not the Youth Settings, after changing to config.before_initialize it worked again.

In Youth Wagon we enable people.people_managers.self_service_managed_creation that should be disabled and overwritten in the SAC Wagon

Largo commented 2 weeks ago

Ich kann bestätigen, dass die Settings.yml vom youth wagon später geladen wird und daher die Einstellungen überschreibt. Im SAC Wagon sind wir auch von initializer "youth.add_settings" zu config.before_initialize gewechselt. Daher ist es sicher notwendig etwas zu machen. Im wagon code sehe ich keine spezielle Abhandlung von wagonname.add_settings, daher koennen wir diese pragmatische Loesung mergen.

Fuer @codez zum visieren.

tobiasstern commented 1 week ago

Hallo @njaeggi was ist hier der Stand? Wurde das abgeschlossen?

Auf der SAC-Integration gibt es dieses Feature noch, vgl Bild:

image
njaeggi commented 1 week ago

Hallo @tobiasstern

Wenn ich auf der Integration auf dem gleichen Kurs versuche ein Kind anzumelden, existiert das Dropdown nicht mehr :thinking: image

Verwendeter Kurs: https://sac-cas.puzzle.ch/de/groups/1/events/18.html

Gibt es andere Bedingungen welche erfüllt sein müssen, damit man ein Kind anmelden kann?

tobiasstern commented 1 week ago

Ja, bei mir kommt die Option nun auch nicht mehr :)

ThomasEllenberger commented 20 hours ago

Könnte das am Featuretoggle liegen? Wir haben im Youth Umgebungen bei denen User sich selbst neue Kinder hinzufügen dürfen, wie beim Verhalten welches ursprünglich von @tobiasstern gemeldet wurde. Und umgebungen bei denen man sich selbst keine neuen Kinder hinzufügen darf, sondern nur solche welche verknüpft sind verwalten darf, was dann bei der Kursanmeldung kein Dropdown mehr hat....

@njaeggi @tobiasstern Kann dieser PR geschlossen werden?