Skip to content

Added docs for JsonResponse::fromJsonString #7204

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

Closed
wants to merge 3 commits into from

Conversation

javiereguiluz
Copy link
Member

This fixes #7174.

@xabbuh
Copy link
Member

xabbuh commented Dec 3, 2016

👍

Status: Reviewed

// if the data to send is already encoded in JSON
$response = JsonResponse::fromJsonString('{ "data": 123 }');

.. versionadded:: 3.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrong version

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, if it is in 3.2, the wrong number is on the next line saying 3.3 (I haven't checked the code to know which version included it)

Copy link
Member

@xabbuh xabbuh Dec 10, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stof
Copy link
Member

stof commented Dec 9, 2016

Status: needs work

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@HeahDude HeahDude added this to the 3.2 milestone Apr 1, 2017
@xabbuh
Copy link
Member

xabbuh commented Apr 15, 2017

Thank you Javier.

xabbuh added a commit that referenced this pull request Apr 15, 2017
…luz)

This PR was squashed before being merged into the 3.2 branch (closes #7204).

Discussion
----------

Added docs for JsonResponse::fromJsonString

This fixes #7174.

Commits
-------

e9a3be0 Added docs for JsonResponse::fromJsonString
xabbuh added a commit that referenced this pull request Apr 15, 2017
@xabbuh xabbuh closed this Apr 15, 2017
weaverryan added a commit that referenced this pull request Apr 15, 2017
* 3.2: (71 commits)
  Rewriting the service container docs
  Minor reword
  Adding a tip for validation in forms without class
  [#7217] add versionadded directives
  [#7203] merge note and versionadded directive
  Use the new configurator YAML syntax
  Added a note about the .htaccess files included by Symfony apps
  Made unmapped field example in forms chapter more descriptive
  [#7507] fix namespace
  [#7507] fix component name
  [#7490] minor typo fix
  Added a note about redirections to absolute URLs in tests
  [#7204] link to API doc
  Added docs for JsonResponse::fromJsonString
  Added the changes suggested by reviewers
  [#7620] use generate() in PHP templates before 2.8
  Fixed the RST syntax
  Improve example context
  Minor formatting changes
  [#7519] some minor tweaks
  ...
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