-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Cookbook] Add note about possible 404 error on Heroku #5073
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
Conversation
|
||
If you take your first steps on Heroku using a fresh installation of | ||
the Symfony Standard Edition you may run into a 404 page not found error. | ||
This is because the route for `/` is defined by the AcmeDemoBundle, but the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor syntax issue:
`/` should be ``/``
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor syntax issue:
`/app/example` should be ``/app/example``
👍 @bicpi thanks for this improvement. |
@javiereguiluz Fixed the backticks, thanks for the hint. |
.. caution:: | ||
|
||
If you take your first steps on Heroku using a fresh installation of | ||
the Symfony Standard Edition you may run into a 404 page not found error. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[...] Standard Edition, you may [...]
I like this one. However, this is all going to change (acmedemobundle will be removed in favor of a symfony demo application and the appbundle will default to /). But I think we should merge this PR soon, to avoid further confusion, and see what's going to happen in the future. |
@wouterj I totally agree with you: merge it ASAP to avoid confusion and let's see what happens in the future. |
👍 |
Is the AcmeDemoBundle going to be removed from the 2.3 branch, too |
@xabbuh Thanks! |
Cool - I agree with everything you guys said :). Thanks! |
…icpi) This PR was merged into the 2.3 branch. Discussion ---------- [Cookbook] Add note about possible 404 error on Heroku | Q | A | ------------- | --- | Doc fix? | no | New docs? | no | Applies to | all | Fixed tickets | - I was running into this 404 and thought something went wrong with the doc root setting first. Commits ------- 1af3913 [Cookbook] Add note about possible 404 error
I was running into this 404 and thought something went wrong with the doc root setting first.