Skip to content
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

Update http_fundamentals.rst #5352

Closed
wants to merge 2 commits into from
Closed

Conversation

wouthoekstra
Copy link
Contributor

@xabbuh
Copy link
Member

xabbuh commented Jun 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
Copy link
Contributor Author

You're right. I'll get on to it :)

@xabbuh
Copy link
Member

xabbuh commented Jun 5, 2015

Oh, and we also have a part talking about this at http://symfony.com/doc/current/cookbook/routing/method_parameters.html#faking-the-method-with-method. Could you please address that section too?

@1emming
Copy link

1emming commented Jun 5, 2015

👍 @wouthoekstra for taking care of this :)

Added some nuance to note about PUT and DELETE in modern browsers.
@xabbuh
Copy link
Member

xabbuh commented Jul 9, 2015

This looks good! 👍

@xabbuh
Copy link
Member

xabbuh commented Jul 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
Copy link
Contributor Author

Ah thx for the tip xabbuh, didn't know that! Glad the alterations are okay :)

weaverryan added a commit that referenced this pull request Jul 12, 2015
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.rst

Modern browsers do support PUT and DELETE, see for example http://stackoverflow.com/questions/165779/are-the-put-delete-head-etc-methods-available-in-most-web-browsers.

Commits
-------

1bc5228 Updated text about HTTP methods
46f3665 Update http_fundamentals.rst
weaverryan added a commit that referenced this pull request Jul 12, 2015
@weaverryan
Copy link
Member

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!

@weaverryan weaverryan closed this Jul 12, 2015
@wouthoekstra
Copy link
Contributor Author

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.

weaverryan added a commit that referenced this pull request Jul 12, 2015
* 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
weaverryan added a commit that referenced this pull request Jul 12, 2015
* 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
weaverryan added a commit that referenced this pull request Jul 12, 2015
* 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
  ...
weaverryan added a commit that referenced this pull request Jul 12, 2015
* 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
Copy link

teameh commented Jul 14, 2015

Nice! @weaverryan you've just merged @wouthoekstra's first Pull! 👍 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants