search

linuxserver / docker-projectsend

GNU General Public License v3.0
74 stars 26 forks source link

Adapt startup script to changes in ProjectSend r1053 #9

Closed tomkuba closed 5 years ago

tomkuba commented 6 years ago

ProjectSend changes in r1053 prevent user from accessing make-config.php page if the configuration exists (even if it is incomplete). Incomplete configuration file is reason why PS refuses to allow user to continue in installation (#7 ). This PR allows user to configure database connection using docker environment variables and creates valid configuration file for ProjectSend. User does not need to configure DB connection on web and can right start with PS configuration wizard.

LinuxServer-CI commented 6 years ago

Can an admin please whisper to jenkins to test this PR.

CHBMB commented 5 years ago

Tested this and can confirm it's working as expected with new installs. It needs rebasing on current master though and readme-vars.yml needs editing. We should also add a note that a database needs to be present already, (It is not automatically created any longer).

CHBMB commented 5 years ago

I've submitted a PR to @tomkuba for some changes, but we have a branch here on our repo.

Apologies for the time it's taken us to get to this.

CHBMB commented 5 years ago

Closing. See https://github.com/linuxserver/docker-projectsend/pull/12#issuecomment-501262833