Uh oh!
There was an error while loading. Please reload this page.
config: change default config for Auto Routing Improved - #8981
Conversation
cd8bd92 to
5dd958eCompare9c00f98 to
61ae88eCompare👋 Hi, @kenjis! |
Feature::$autoRoutesImproved -> true Routing::$translateUriToCamelCase -> true
61ae88e to
ecfbc2fComparekenjis
commented
Jun 24, 2024
|
Even if you don't change the default controller name, you must remove
the defined route `$routes->get('/', 'Home::index');`.kenjis
commented
Jun 24, 2024
I have made a few improvements. |
| Examples of Controller Names | ||
| **************************** | ||
| This is an valid controller name. Because ``App\Controllers\HelloWorld`` is CamelCase. |
There was a problem hiding this comment.
Honestly, I didn't like using 'This is', it made the review difficult for me.
There was a problem hiding this comment.
I changed to The following is.
| 1. The first segment represents the controller **class** that should be invoked. | ||
| 2. The second segment represents the class **method** that should be called. | ||
| 3. The third, and any additional segments, represent any **parameters** that will be passed to the controller. |
There was a problem hiding this comment.
I took from this sentence that the parameters are sent to the controller.
But the parameters are sent to the methods in the controller.
There was a problem hiding this comment.
Changed to the controller method.
neznaika0
commented
Jun 25, 2024
I think it's worth adding a note about Filters. |
kenjis
commented
Jun 25, 2024
@neznaika0 Added. |
Needs #8980
Description
Feature::$autoRoutesImproved-> trueRouting::$translateUriToCamelCase-> trueChecklist: