Skip to content

[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

Merged
merged 1 commit into from
Apr 3, 2015
Merged

[Cookbook] Add note about possible 404 error on Heroku #5073

merged 1 commit into from
Apr 3, 2015

Conversation

bicpi
Copy link
Contributor

@bicpi bicpi commented Mar 11, 2015

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.


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
Copy link
Member

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 ``/``

Copy link
Member

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``

@javiereguiluz
Copy link
Member

👍 @bicpi thanks for this improvement.

@bicpi
Copy link
Contributor Author

bicpi commented Mar 11, 2015

@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.
Copy link
Member

Choose a reason for hiding this comment

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

[...] Standard Edition, you may [...]

@wouterj
Copy link
Member

wouterj commented Mar 25, 2015

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.

@javiereguiluz
Copy link
Member

@wouterj I totally agree with you: merge it ASAP to avoid confusion and let's see what happens in the future.

@xabbuh
Copy link
Member

xabbuh commented Mar 25, 2015

👍

@bicpi
Copy link
Contributor Author

bicpi commented Mar 26, 2015

Is the AcmeDemoBundle going to be removed from the 2.3 branch, too
Sorry, I missed the discussion about this step.

@xabbuh
Copy link
Member

xabbuh commented Mar 26, 2015

@bicpi
Copy link
Contributor Author

bicpi commented Mar 26, 2015

@xabbuh Thanks!

@weaverryan
Copy link
Member

Cool - I agree with everything you guys said :). Thanks!

@weaverryan weaverryan merged commit 1af3913 into symfony:2.3 Apr 3, 2015
weaverryan added a commit that referenced this pull request Apr 3, 2015
…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
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.

5 participants