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

Change settings add source to before initialize #89

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

njaeggi
Copy link
Contributor

@njaeggi njaeggi commented Oct 31, 2024

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

@njaeggi njaeggi assigned njaeggi and unassigned njaeggi Oct 31, 2024
@Largo
Copy link
Contributor

Largo commented Oct 31, 2024

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.

@Largo Largo merged commit df56ad9 into master Oct 31, 2024
9 checks passed
@Largo Largo deleted the fix-feature-gate-override branch October 31, 2024 19:32
@Largo Largo self-requested a review October 31, 2024 19:32
Copy link
Contributor

@Largo Largo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Largo Largo assigned Largo and unassigned Largo Oct 31, 2024
@tobiasstern
Copy link

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

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

@njaeggi
Copy link
Contributor Author

njaeggi commented Nov 11, 2024

Hallo @tobiasstern

Wenn ich auf der Integration auf dem gleichen Kurs versuche ein Kind anzumelden, existiert das Dropdown nicht mehr 🤔
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
Copy link

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

@ThomasEllenberger
Copy link

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?

@tobiasstern
Copy link

Nach mir kann das geschlossen werden, ja

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants