-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[HttpClient] Add HttpOptions->addHeader as a shortcut to add an header … #19236
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I'd be happy to help here as well, but on the Http-Client documentation, there is no mention of the HttpOptions object at all. |
Hi @Dean151. Congratz with the code PR merge :) I think it makes sense to document this nice helper class. We can add a little paragraph to https://symfony.com/doc/current/http_client.html#configuration saying like "There is this nice helper that you can use to build up the options array". And then show an example (maybe even copy the existing code example and provide the HttpOptions alternative). Then, I don't think we need to document all available methods on this class in the documentation: we can just link to the class and let people discover all methods themselves (or use their autocompletion). Are you willing to do this task? (happy to guide you) |
I'd be happy to write a small generic paragraph on HttpOptions on this page. |
In the docs, we're following the "maintained versions" from Symfony's release policy. So it would be great if you submit this to the 5.4 branch (the oldest maintained branch). |
…t (Dean151) This PR was merged into the 5.4 branch. Discussion ---------- [HttpClient] Added a paragraph about HttpOptions object Partial fix of #19236 A note will be added on the 7.1 branch after this gets merged to warn about the override behavior of ->setHeaders instead of newly added ->addHeader. Commits ------- dbc790b Added a paragraph about HttpOptions object
Fixes symfony#19236 Not sure of the documentation format. Open to suggestion to improve it :)
Fixes symfony#19236 Not sure of the documentation format. Open to suggestion to improve it :)
We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.
To fix this issue, please create a PR against the 7.1 branch in the symfony-docs repository.
Thank you! 😃
The text was updated successfully, but these errors were encountered: