Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Updates to Platform.sh deploy instructions#8397
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
* Update example PHP version from 5.6 to 7.1* Files in `.platform` are now required* Endpoint name `php` is better written as `http`* Build flavor `symfony` is better written as `composer`* Rename database from `mysql` to `mydatabase` to separate it clearly from the service type* The `web` section should now use `locations`
| type:php:7.1 | ||
| build: | ||
| flavor:symfony | ||
| flavor:composer |
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.
why do you change this?symfony is correct
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.
I read somewhere thatcomposer is the new and recommended type for all PHP. Their Symfony template doc use that too:https://docs.platform.sh/frameworks/symfony.html
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.
good to know, thank you@javiereguiluz 👍
| type:upstream | ||
| # the first part should be your project name | ||
| upstream:'myphpproject:php' | ||
| upstream:'myphpproject:http' |
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.
should be:
'https://{default}/':type:upstreamupstream:"myphpproject:http"'https://www.{default}/':type:redirectto:"https://{default}/"
OskarStark commentedOct 6, 2017
Refs#8483 |
| bin/console --env=prod cache:clear | ||
| For best practices, you shouldalso add a ``.platform`` folder at the root of | ||
| Youalso need add a ``.platform`` folder at the root of |
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.
to add
javiereguiluz commentedNov 28, 2017
@pjcdawkins thanks for this contribution, but sadly we must close this pull request without merging it. Why? Because in#8721 we decided to remove all these articles explaining how to deploy to third-party services. We now simply link to the articles published by those services. Thanks for understanding it. |
Uh oh!
There was an error while loading.Please reload this page.
.platformare now requiredphpis better written ashttpsymfonyis better written ascomposermysqltomydatabaseto separate it clearly from the service typewebsection should now uselocationsThere are probably additional improvements that can be made, referring to the Platform.shdocumentation and to the work that has been done inhttps://github.com/platformsh/platformsh-example-symfony/