Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
javiereguiluz wants to merge3 commits intosymfony:3.2fromjaviereguiluz:fix_7174

Conversation

@javiereguiluz
Copy link
Member

Thisfixes#7174.

@xabbuh
Copy link
Member

👍

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
MemberAuthor

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)

javiereguiluz reacted with thumbs up emoji
Copy link
Member

@xabbuhxabbuhDec 10, 2016
edited
Loading

Choose a reason for hiding this comment

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

javiereguiluz and HeahDude reacted with thumbs up emoji
@stof
Copy link
Member

stof commentedDec 9, 2016

Status: needs work

Copy link
Contributor

@HeahDudeHeahDude left a comment

Choose a reason for hiding this comment

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

👍

@HeahDudeHeahDude added this to the3.2 milestoneApr 1, 2017
@xabbuh
Copy link
Member

Thank you Javier.

xabbuh added a commit that referenced this pull requestApr 15, 2017
…luz)This PR was squashed before being merged into the 3.2 branch (closes#7204).Discussion----------Added docs for JsonResponse::fromJsonStringThisfixes#7174.Commits-------e9a3be0 Added docs for JsonResponse::fromJsonString
xabbuh added a commit that referenced this pull requestApr 15, 2017
@xabbuhxabbuh closed thisApr 15, 2017
weaverryan added a commit that referenced this pull requestApr 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 freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@xabbuhxabbuhxabbuh left review comments

+2 more reviewers

@stofstofstof left review comments

@HeahDudeHeahDudeHeahDude approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

3.2

Development

Successfully merging this pull request may close these issues.

5 participants

@javiereguiluz@xabbuh@stof@HeahDude@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp