- Issue created by @Shifali Baghel
- 🇮🇳India Shifali Baghel Gurgaon
Hi Josh Waihi @fiasco, hope you are doing well.
I saw a few things, and the problem was with the two YML files for a single form. In my view, each configuration form should have its own configuration file. Could this separate module's configuration (http_cache_control.settings.yml) be moved to this "system.performance.yml" file?
Please share your opinions with me on this.Thank you
- 🇳🇿New Zealand Josh Waihi
Hi @Shifali Baghel, I don't quite understand why we need to move the settings to under system.performance. It seems cleaner to keep the module's configuration under its own namespace to prevent collisions with other modules and mitigate potential core bugs using another module's namespace.
- Status changed to Closed: works as designed
11 months ago 9:30am 17 January 2024 - 🇧🇪Belgium dieterholvoet Brussels
Exactly, a contrib module shouldn't add extra config options to core config.