-
-
Notifications
You must be signed in to change notification settings - Fork 812
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
Additional boot.settings to enable running Standalone without a civicrm.settings.php #31155
base: master
Are you sure you want to change the base?
Conversation
…e loading from env/default
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
16fa691
to
c382348
Compare
@ufundo - I tried this out as follows:
I wasn't sure where /var/www/html/setting-stub should exist or not so I removed it but got the same message. I also tried passing CIVICRM_DSN instead of, and as well as CIVICRM_DB_* variables but but got the same message. Let me know if I am missing something or you want me to test again. |
Sorry @michaelmcandrew that doesnt exist as an env var (yet) For now I got it working by adding
at the top of civicrm.standalone.php |
I can confirm that adding that line works for me :) I was able to log in successfully. |
Overview
Following on from #30533 - this brings a few more constants into the
*.boot.setting.php
system, in order that they can be loaded from environment variables or defaults.It's based upon #31150 - between the two I was able to run Standalone without civicrm.settings.php - just providing a three env vars.
Before
civicrm.settings.php
provides many critical constantsAfter
CIVICRM_DSN
,CIVICRM_UF_BASEURL
andCIVICRM_UF
as environment variablescivicrm.settings.php
Comments
It's running Smarty2, but it's running....
@michaelmcandrew could you have a go testing this branch with the Docker container?