Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Update http_fundamentals.rst#5352
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
xabbuh commentedJun 3, 2015
We should probably explain that all modern browsers support these methods in XHR, but that HTML forms still can only use GET and POST. |
wouthoekstra commentedJun 4, 2015
You're right. I'll get on to it :) |
xabbuh commentedJun 5, 2015
Oh, and we also have a part talking about this athttp://symfony.com/doc/current/cookbook/routing/method_parameters.html#faking-the-method-with-method. Could you please address that section too? |
1emming commentedJun 5, 2015
👍@wouthoekstra for taking care of this :) |
Added some nuance to note about PUT and DELETE in modern browsers.
xabbuh commentedJul 9, 2015
This looks good! 👍 |
xabbuh commentedJul 9, 2015
@wouthoekstra By the way, we don't receive notifications when new commits are pushed to a pull request (I just discovered your update "accidentally"). So next time it's better to leave a comment after you did some changes. :) |
wouthoekstra commentedJul 9, 2015
Ah thx for the tip xabbuh, didn't know that! Glad the alterations are okay :) |
This PR was submitted for the 2.8 branch but it was merged into the 2.3 branch instead (closes#5352).Discussion----------Update http_fundamentals.rstModern browsers do support PUT and DELETE, see for examplehttp://stackoverflow.com/questions/165779/are-the-put-delete-head-etc-methods-available-in-most-web-browsers.Commits-------1bc5228 Updated text about HTTP methods46f3665 Update http_fundamentals.rst
weaverryan commentedJul 12, 2015
I like it very much - nice work@wouthoekstra! I've merged this into the 2.3 branch (and we'll merge to other branches from there) and added a minor tweak at sha:f9ef3ee Cheers! |
wouthoekstra commentedJul 12, 2015
Sounds good! Sorry, I understood only after my pull request that I should have made the request on the 2.3 branch, not on the latest one. |
* 2.3: [#5352] minor language tweak Updated text about HTTP methods Update http_fundamentals.rst [HttpKernel] Fix use statement Fixes small typo in data transformers cookbook some tweaks to the data transformers chapter Updated the Symfony Versions Roadmap image Fixed a minor typo Fixed minor issues Removed some information wrongly rebased Style / grammar fixes for the deployment chapter Style / grammar fixes for the controller chapter Style / grammar fixes for the console chapter Removes wrong comma Rewritten 'whether' & removed serial comma Style / grammar fixes for the configuration chapter Style / grammar fixes for the cache chapter Style / grammar fixes for the bundles chapter Style / grammar fixes for the assetic chapter
* 2.6: [#5352] minor language tweak Updated text about HTTP methods Update http_fundamentals.rst [HttpKernel] Fix use statement Fixes small typo in data transformers cookbook some tweaks to the data transformers chapter Updated the Symfony Versions Roadmap image Fixed a minor typo Fixed minor issues Removed some information wrongly rebased Style / grammar fixes for the deployment chapter Style / grammar fixes for the controller chapter Style / grammar fixes for the console chapter Removes wrong comma Rewritten 'whether' & removed serial comma Style / grammar fixes for the configuration chapter Style / grammar fixes for the cache chapter Style / grammar fixes for the bundles chapter Style / grammar fixes for the assetic chapter
* 2.7: (21 commits) [#5352] minor language tweak Updated text about HTTP methods Update http_fundamentals.rst [HttpKernel] Fix use statement [PSR-7] Fix Diactoros link Fixes small typo in data transformers cookbook some tweaks to the data transformers chapter Updated the Symfony Versions Roadmap image Fixed a minor typo Fixed minor issues Use the built-in serializer instead of promoting JMS Removed some information wrongly rebased Style / grammar fixes for the deployment chapter Style / grammar fixes for the controller chapter Style / grammar fixes for the console chapter Removes wrong comma Rewritten 'whether' & removed serial comma Style / grammar fixes for the configuration chapter Style / grammar fixes for the cache chapter Style / grammar fixes for the bundles chapter ...
* 2.8: (21 commits) [#5352] minor language tweak Updated text about HTTP methods Update http_fundamentals.rst [HttpKernel] Fix use statement [PSR-7] Fix Diactoros link Fixes small typo in data transformers cookbook some tweaks to the data transformers chapter Updated the Symfony Versions Roadmap image Fixed a minor typo Fixed minor issues Use the built-in serializer instead of promoting JMS Removed some information wrongly rebased Style / grammar fixes for the deployment chapter Style / grammar fixes for the controller chapter Style / grammar fixes for the console chapter Removes wrong comma Rewritten 'whether' & removed serial comma Style / grammar fixes for the configuration chapter Style / grammar fixes for the cache chapter Style / grammar fixes for the bundles chapter ...
teameh commentedJul 14, 2015
Nice!@weaverryan you've just merged@wouthoekstra's first Pull! 👍 😀 |
Modern browsers do support PUT and DELETE, see for examplehttp://stackoverflow.com/questions/165779/are-the-put-delete-head-etc-methods-available-in-most-web-browsers.