- Issue created by @mondrake
- last update
over 1 year ago Custom Commands Failed - @mondrake opened merge request.
- last update
over 1 year ago 30,069 pass, 78 fail - last update
over 1 year ago Custom Commands Failed - last update
over 1 year ago 30,073 pass, 76 fail - last update
over 1 year ago 30,144 pass, 2 fail - Status changed to Needs review
over 1 year ago 6:08pm 10 September 2023 - last update
over 1 year ago 30,145 pass - last update
over 1 year ago 30,145 pass - last update
over 1 year ago 30,145 pass - Status changed to Needs work
over 1 year ago 6:31pm 13 September 2023 - 🇺🇸United States smustgrave
Since that change record is already published, could a new one be written? And links updated.
Thoughts?
- Status changed to Needs review
over 1 year ago 9:28pm 13 September 2023 - 🇮🇹Italy mondrake 🇮🇹
Before talking CR, can we have a review on what is actually suggested as a change?
- 🇺🇸United States smustgrave
Think the submaintainer could be the one to make that call.
- 🇳🇱Netherlands daffie
Would it not be better to, when the module name is not set, to let it default to the database drivers name? Instead of require existing sites to add the module name. That will result in a lot of communication to existing site owners.
Having the module name default to the database driver name will result that you only have to set the module name when it is not the same as the database drivers name. For >99% of the existing sites is the module name the same as the database drivers name.
- Status changed to Needs work
about 1 year ago 4:11pm 28 September 2023