search

polydigital / backdrop-civicrm-docker

0 stars 1 forks source link

Auto install civicrm database credentials #7

Closed tabroughton closed 5 years ago

tabroughton commented 5 years ago

When deploying a fresh docker we need to auto install the civicrm module so that the db is set up with the credentials, etc.

tabroughton commented 5 years ago

I'm not keen on the way that the current drush install works for civicrm.

  1. the civicrm module needs enabling before the drush commands are available but that causes errors
  2. putting the drush commands in .drush/commands complains that backdrop isn't yet ready for the command
  3. the code seems to replicate what the core install code does (though I'd need a closer look)
  5. When installed the theme isn't working properly
  6. the loading generated data flag true cases a db error on install
tabroughton commented 5 years ago

1 and 2 are solved though it would be good to have another way of including the civicrm command to the backdrop.drush.inc file (maybe through something in entrypoint directly editing the file).

will create a new ticket for that

tabroughton commented 5 years ago

loading generated data fails on auto install, this is a major issue for me

tabroughton commented 5 years ago

5 is fixed by adding the port number to the site_url on the install command

tabroughton commented 5 years ago

6 is caused by an error in the installation code of civicrm-backdrop drush command, a missing if statement. Have a fix and made PR, tracking this in #9

tabroughton commented 5 years ago
  1. we will need to monitor changes to civicrm-core install and make sure they are reflected in drush command, I've created a ticket to consider this
tabroughton commented 5 years ago
  1. Needs a fix for permissions in upstream dependency. I've created a ticket for this.
tabroughton commented 5 years ago

All concerns have been covered with other tickets