- Issue created by @joachim
- 🇮🇳India dineshkumarbollu
Hi @joachim
Based on OptionsRequestSubscriber class methods added below docs, Is this lines are good.
* This event subscriber listens to KernelEvents::REQUEST and responds to OPTIONS requests * by providing an Allow header listing all the HTTP methods allowed for the requested routes. - 🇬🇧United Kingdom joachim
Looks good, though you can drop the initial 'This event subscriber'.
- Merge request !7536Class docs change for OptionsRequestSubscriber → (Closed) created by dineshkumarbollu
- 🇳🇿New Zealand quietone
Just stopping in to say this needs a title that is descriptive of the fix.
- Status changed to RTBC
7 months ago 8:34am 17 April 2024 - 🇬🇧United Kingdom joachim
MR looks good.
The title describes the problem, so I'm not sure what's wrong with it.
-
longwave →
committed 398192b2 on 10.2.x
Issue #3436136 by dineshkumarbollu, joachim: Improve documentation for...
-
longwave →
committed 398192b2 on 10.2.x
- 🇬🇧United Kingdom longwave UK
Backported to 10.2.x as a docs-only bug fix.
Committed and pushed 4b54857210 to 11.x and 7506af29b5 to 10.3.x and 398192b2d0 to 10.2.x. Thanks!
-
longwave →
committed 7506af29 on 10.3.x
Issue #3436136 by dineshkumarbollu, joachim: Improve documentation for...
-
longwave →
committed 7506af29 on 10.3.x
- Status changed to Fixed
7 months ago 1:40pm 17 April 2024 -
longwave →
committed 4b548572 on 11.x
Issue #3436136 by dineshkumarbollu, joachim: Improve documentation for...
-
longwave →
committed 4b548572 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.