-
Notifications
You must be signed in to change notification settings - Fork 691
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
Configuring Settings.php D8 Doc Update #918
Comments
@timani it looks like you've done most of this already here: https://github.com/pantheon-systems/pantheon-settings-examples/tree/master/drupal-8 |
Oh right, I remember starting this but it does need an update to be complete |
Also, as a 'hash_salt' setting is needed, should we include a method for users to add that to their settings.php for their local environment? We set it per environment with the following code:
|
#1097 was created as a duplicate - adding related description here. @timani are you planning a PR for this issue? Issue Description:Drupal 8 includes a bundled settings.php file, unlike previous versions. Suggested ResolutionUpdate doc throughout as needed to align with Drupal 8, while preserving suggestions/best practices already mentioned that apply to D6/D7.
|
Yes, folks should make their own HASH_SALT value in their settings.local.php file. See: https://github.com/pantheon-systems/drush-config-workflow/blob/master/bin/quickstart If for some reason someone wants to use the Pantheon HASH_SALT in their local site (not necessary), then can get it via:
|
Closes #918 - Revise intro w/d8 conent - Warn against replacing the bundled settings.php file - States that d8 on pantheon uses an unmodified version of core - Add D8 section on local environment configs, recommends settings.local.php and links to drush quickstart script - Provide Terminus drush command to retrun HASH_SALT value
Re: Configuring Settings.php
Issue Description:
Suggested Resolution
The text was updated successfully, but these errors were encountered: