- Issue created by @Rory Downes
- Status changed to Needs review
11 months ago 8:19pm 22 December 2023 - last update
11 months ago Patch Failed to Apply - Status changed to Needs work
11 months ago 4:13pm 23 December 2023 - 🇺🇸United States smustgrave
Will need test coverage if this lands. But can ask the "why" the need to change it?
- 🇬🇧United Kingdom Rory Downes
Hi,
I have a request from a client to make the message not quite so direct.
Regards Rory - 🇺🇸United States smustgrave
So ban actually doesn't have a submaintainer so will elevate to framework manager.
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
I think this probably belongs in https://www.drupal.org/project/advban → for the following reasons
* We don't have a ban maintainer
* The module is in the hotpath and needs to be as performant as possibleI think it would be simple and performant to make it configurable via settings.php
But config and UI feels too like overkill when anything more advanced is in the contib module
- Status changed to Closed: won't fix
11 months ago 11:19pm 23 December 2023 - 🇫🇷France andypost
There's string overrides specifically for that https://git.drupalcode.org/project/drupal/-/blob/10.2.x/sites/default/de...
String overrides won't help on this one, because the core ban message isn't translated. AdvBan did the trick though.