8000 Update homestead.rst by josedevega · Pull Request #6762 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Update homestead.rst #6762

8000
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 1 commit into from
Closed

Conversation

josedevega
Copy link

Just a tip for people using symfony components in their php projects

Just a tip for people using symfony components in their php projects
@xabbuh
Copy link
Member
xabbuh commented Jul 22, 2016

What is the difference when switching to this type?

@josedevega
Copy link
Author

Hi

Provisioning of /etc/nginx/sites-available file, when type is symfony,*.php
requests return the php sourcecode.

I am sending both .sh files

Regards

El 22 de julio de 2016 a las 22:45 Christian Flothmann
notifications@github.com escribió:

What is the difference when switching to this type?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub

#6762 (comment) , or
mute the thread
https://github.com/notifications/unsubscribe-auth/AQBrtvyjknHYDtn4vGX6YqgrHu64CxVBks5qYSvegaJpZM4JSp42
.

@javiereguiluz
Copy link
Member

@josedevega thanks for proposing this improvement. We recently changed the entire Symfony Docs, so this pull request can no longer be merged. I'm sorry. I've recreated your changes in #6831.

@josedevega
Copy link
Author

Hi @javiereguiluz

Thanks for recreating it, I think it is important for the comunity.

Regards

El 3 de agosto de 2016 a las 10:38 Javier Eguiluz notifications@github.com
escribió:

@josedevega https://github.com/josedevega thanks for proposing this

improvement. We recently changed the entire Symfony Docs
http://symfony.com/blog/introducing-the-new-symfony-documentation , so this
pull request can no longer be merged. I'm sorry. I've recreated your changes
in #6831 #6831 .

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub

#6762 (comment) , or
mute the thread
https://github.com/notifications/unsubscribe-auth/AQBrtn2JJ0svaO0rqIXCLdoTWONlY5H5ks5qcFODgaJpZM4JSp42
.

@xabbuh
Copy link
Member
xabbuh commented Aug 3, 2016

Thank you for starting this @josedevega!

@xabbuh xabbuh closed this Aug 3, 2016
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.

4 participants
0