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

[7.0] Update & Simplify install instructions #2167

Merged
merged 4 commits into from Nov 28, 2017

Conversation

4 participants
@andrerom
Member

andrerom commented Nov 27, 2017

Now that we require PHP 7.1 and have greatly improved performance of backend placing hinst on performance tuning does not need to take space here anymore.

Besides that:

  • updated to use dev, with symfony 3 builtin server is only available in dev by default
  • update to use /admin url for backend
  • somewhat simplified the text a bit

Open question:

  • Do we need to run the assetic:dump command? As it's already ran by composer create-project which is what has been done when this is shown, ok if we remove?
[7.0] Update install instructions
Now that we require PHP 7.1 and have greatly improved performance of backend placing hinst on performance tuning does not need to take space here anymore.

Besides that:
- updated to use dev, with symfony 3 builtin server is only available in dev by default
- update to use `/admin` url for backend
- somewhat simplified the text a bit

Open question:
- Do we need to run the assetic:dump command? As it's already ran by composer create-project which is what has been done when this is shown, ok if I remove?

@andrerom andrerom changed the title from [7.0] Update install instructions to [7.0] Update & Simplify install instructions Nov 27, 2017

@alongosz

+1, one nitpick:

andrerom added some commits Nov 28, 2017

@andrerom

This comment has been minimized.

Show comment
Hide comment
@andrerom

andrerom Nov 28, 2017

Member

Ok, fixed comments and simplified a bit more. I'm assuming the assetic:dump was a copy paste from install doc which had it for when people install from archive and need to setup symlinks again.

review ping @alongosz @DominikaK @SylvainGuittard

Member

andrerom commented Nov 28, 2017

Ok, fixed comments and simplified a bit more. I'm assuming the assetic:dump was a copy paste from install doc which had it for when people install from archive and need to setup symlinks again.

review ping @alongosz @DominikaK @SylvainGuittard

@andrerom andrerom merged commit 555a8b7 into 7.0 Nov 28, 2017

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@andrerom andrerom deleted the 7.0-install-instructions branch Nov 28, 2017

wizhippo added a commit to wizhippo/ezpublish-kernel that referenced this pull request Apr 12, 2018

[7.0] Update & Simplify install instructions (ezsystems#2167)
* [7.0] Update install instructions

Now that we require PHP 7.1 and have greatly improved performance of backend placing hinst on performance tuning does not need to take space here anymore.

Besides that:
- updated to use dev, with symfony 3 builtin server is only available in dev by default
- update to use `/admin` url for backend
- somewhat simplified the text a bit

Open question:
- Do we need to run the assetic:dump command? As it's already ran by composer create-project which is what has been done when this is shown, ok if I remove?

* Update ScriptHandler.php

* Update ScriptHandler.php

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