git clone git@github.com:Backdrop4Good/backdrop4good.org.git
If that doesn't work, try the HTTPS URL instead of SSH: git clone https://github.com/Backdrop4Good/backdrop4good.org.git
settings.php
Current:
Add in a settings.php to the web directory. The file that points to the
config (which is outside of the web directory).
There was already a settings.php in the directory, and the downloaded file was renamed to settings.php.1. Should we simply start the text with something like "Remove the settings.php from the web directory", and then: add the file from Github and so on as described above?
lando start
Current:
lando start
Comment:
I was not sure if I have to type that command from the web directory. I even forgot if I did it and if it was succesful, I guess yes. I suggest however to add instructions like cd xyz anywhere in the README, even in cases where it might be implicetly clear for experienced developers if they are already in the correct directory or not.
Other
When I went to the site in the browser, I got an exception saying that the '../config/active' directory is empty or missing crucial files. I checked it and saw the directory wasn't there. Suggestions in Gitter to fix were: (a) type lando drush bcim in web, or (b) create the missing directory and copy all files from the existing directory config/staging to the created config/active. While (b) worked, (a) didn't.
Not sure if this is a bug or a documentation issue.
When I spinned up the Backdrop4Good site locally as described in https://github.com/Backdrop4Good/backdrop4good.org/blob/master/README.md, I noticed some issues which can be fixed improving the instructions. Here are a few suggestions and some comments:
git clone
Current:
git clone git@github.com:Backdrop4Good/backdrop4good.org.gitSuggestion:
git clone git@github.com:Backdrop4Good/backdrop4good.org.gitIf that doesn't work, try the HTTPS URL instead of SSH:
git clone https://github.com/Backdrop4Good/backdrop4good.org.gitsettings.php
Current:
settings.phpto thewebdirectory. The file that points to the config (which is outside of thewebdirectory).wget https://github.com/backdrop/backdrop/raw/1.x/settings.phpComment:
There was already a
settings.phpin the directory, and the downloaded file was renamed tosettings.php.1. Should we simply start the text with something like "Remove thesettings.phpfrom thewebdirectory", and then: add the file from Github and so on as described above?lando start
Current:
lando startComment:
I was not sure if I have to type that command from the
webdirectory. I even forgot if I did it and if it was succesful, I guess yes. I suggest however to add instructions likecd xyzanywhere in the README, even in cases where it might be implicetly clear for experienced developers if they are already in the correct directory or not.Other
When I went to the site in the browser, I got an exception saying that the '../config/active' directory is empty or missing crucial files. I checked it and saw the directory wasn't there. Suggestions in Gitter to fix were: (a) type
lando drush bciminweb, or (b) create the missing directory and copy all files from the existing directoryconfig/stagingto the createdconfig/active. While (b) worked, (a) didn't.Not sure if this is a bug or a documentation issue.