Skip to content

Add documentation for the new BinaryFileResponse class (#1866)#2416

Merged
weaverryan merged 3 commits intosymfony:2.2from
jbafford:i1866
Apr 4, 2013
Merged

Add documentation for the new BinaryFileResponse class (#1866)#2416
weaverryan merged 3 commits intosymfony:2.2from
jbafford:i1866

Conversation

@jbafford
Copy link
Copy Markdown
Contributor

Q A
Doc fix? yes
New docs? yes (symfony/symfony#4546)
Applies to 2.2
Fixed tickets #1866

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please add an empty line before this one

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A little typo here: I think you meant Symfony\\Component\\HttpFoundation\\BinaryFileResponse

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You're correct. I have that fixed now.

weaverryan added a commit that referenced this pull request Apr 4, 2013
Add documentation for the new BinaryFileResponse class (#1866)
@weaverryan weaverryan merged commit 4140805 into symfony:2.2 Apr 4, 2013
@weaverryan
Copy link
Copy Markdown
Contributor

Hey John!

This is really great - and I get to learn about a new feature from your work ;). I've only made a few small changes and also mentioned X-Sendfile at sha: 615a7b3. If you see any issues, let me know!

Thanks!

@jbafford
Copy link
Copy Markdown
Contributor Author

jbafford commented Apr 4, 2013

Glad to be of help! :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants