Skip to content

Add new symfony4 site type to homestead documentation #8078

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

michaelesmith
Copy link
Contributor

A new site type was added in laravel/homestead#600 for index.php only configurations. This pr adds this information to the homestead documentation.

The ``type`` option tells Homestead to use the Symfony nginx configuration.
The ``type`` option tells Homestead to use the Symfony nginx configuration.
Homestead now supports a Symfony2 web layout with app.php and app_dev.php when
using type: symfony2 and an index.php layout when using type: symfony4.
Copy link
Member

@xabbuh xabbuh Jun 22, 2017

Choose a reason for hiding this comment

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

What happens when the symfony layout (with neither 2 nor 4) is used as it was before?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When symfony is used it defaults to serve_symfony2.sh configuration (this hasn't changed). I added the symfony2 for clarity when discussing the new symfony4 type.

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

The front controller of Symfony 4 has changed. Instead of web/index.php, it'll be public/index.php

@HeahDude HeahDude added this to the 4.0 milestone Jul 29, 2017
wouterj added a commit that referenced this pull request Nov 13, 2017
@wouterj wouterj mentioned this pull request Nov 13, 2017
wouterj added a commit that referenced this pull request Nov 13, 2017
This PR was merged into the master branch.

Discussion
----------

Homestead symfony4

/Continues #8078

Commits
-------

70291d7 [#8078] Added some inline code blocks
2a3660e Add new symfony4 site type to homestead documentation
@wouterj
Copy link
Member

wouterj commented Nov 13, 2017

Hi @michaelesmith!

I've rebased your PR on the 4.0 version of the docs and added some minor inline code blocks in a seperate commit. This is now merged with PR #8640. You'll still receive your credits for your nice work! Thanks for documenting this new homestead feature.

@wouterj wouterj closed this Nov 13, 2017
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.

6 participants