- Issue created by @Anybody
- 🇩🇪Germany Anybody Porta Westfalica
Maybe this is a good novice job, if a novice reads into the README or the code and adds the
#description
using the Form API for this settings form? - Merge request !7Issue #3498575: Added description to the settings form elements. → (Merged) created by divyansh.gupta
- 🇮🇳India divyansh.gupta Jaipur
Hello @anybody,
I have read the Readme file and the functionality and updated the description of fields of settings form.
Please review. - 🇩🇪Germany Anybody Porta Westfalica
Thanks @divyansh.gupta - really nice and helpful additions. I left two comments, maybe a maintainer could also have a look for final review afterwards. Thank you! Really nice!
- 🇩🇪Germany Anybody Porta Westfalica
Thanks @divyansh.gupta! Great work. I think now a maintainer should decide, if all important information is given and if it's all fine!
Let's set this RTBC! :)
- First commit to issue fork.
-
jurgenhaas →
committed 7c762e29 on 1.1.x authored by
divyansh.gupta →
Issue #3498575: Add form description for settings fields
-
jurgenhaas →
committed 7c762e29 on 1.1.x authored by
divyansh.gupta →
- 🇩🇪Germany jurgenhaas Gottmadingen
Those enhancements look good to me, just merged them. Thank you all for your contributions.