old !28 was never merged into master
Retry merging it into master here, so roles can be forwarded from the OIDC client
Activity
Reference to old MR: https://code.greenhost.net/open/wordpress-helm/-/merge_requests/28
created merge request !11 (merged) to address this issue
mentioned in merge request !11 (merged)
- Best practice is to have a separate plugin and to not touch core or to suffix functions.php
- What is an easy and visible way to have a wordpress-helm plugin where these functions can placed
Edited by Rebar-
Best practice is to have a separate plugin and to not touch core or to suffix functions.php
I know, that's why this is off by default. However, we need it for OpenAppStack.
Future work could include making a plugin for this, but it's not our highest priority at the moment.
What is an easy and visible way to have a wordpress-helm plugin where these functions can placed
As far as I know helm does not support making "plugins" for charts.
closed via merge request !11 (merged)
mentioned in commit 07bcc4f0