- 🇩🇪Germany Anybody Porta Westfalica
@yivanov thank you for the nice work!
What's missing I guess is an update hook transfering the config values into the newly created config!
Also could you please create this as MR instead of a patch?Happy to review again, thank you!
- First commit to issue fork.
- 🇩🇪Germany Grevil
Yep, the patch doesn't apply any more. Maybe @yivanov could give a new, adjusted version of the patch as an MR on the current issue fork!
- 🇫🇷France mably
I don't think we need to change the configuration structure, unless I miss something?
Switching from
string
tolabel
or translatabletext
should be enough.And would facilitate its merging.
- Merge request !27Issue #3270697 by yivanov, mably: Add config schema to allow email template... → (Open) created by mably
- 🇫🇷France mably
Just created an MR based on @yivanov patch without changing the configuration structure.
Any feedback is welcome.