- Issue created by @kim.pepper
- First commit to issue fork.
- Status changed to Needs review
8 months ago 4:27pm 30 October 2023 - 🇫🇷France vbouchet
I only noticed the issue's fork when uploading the patch. Seems the fork is empty anyway.
- last update
8 months ago 30,481 pass - Status changed to RTBC
8 months ago 11:03pm 31 October 2023 - 🇺🇸United States smustgrave
Same, believe the deprecation is correct. CR is correct and tests were updated.
LGTM
- 🇦🇺Australia kim.pepper 🏄♂️🇦🇺Sydney, Australia
Is this a valid change? The RouteBuilder is explicitly looking for Controllers so ControllerResolver makes more sense than the more generic CallableResolver imo.
- last update
8 months ago 30,483 pass - last update
8 months ago 30,486 pass - last update
8 months ago 30,486 pass - last update
8 months ago 30,493 pass - last update
8 months ago 30,506 pass, 2 fail - Status changed to Needs work
8 months ago 12:02am 11 November 2023 The Needs Review Queue Bot → tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
