-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Problem with wp-config.php file #573
Comments
Could you just use |
Indeed, |
Hey @swissspidy and @tianon ! I just stumbled over this, and we are using
I just tried a workaround and if I add wordpress/wp-config-docker.php Line 112 in 1d90641
|
Yep, sounds like you're running into what caused WordPress/WordPress@cbcd3ff (which we've applied in 1d90641 but not quite published yet). Given it's breaking WP-CLI, I've opened docker-library/official-images#10227 to get that comment change pushed out. |
Hi,
we were using wp-config-sample.php file to keep some project specific variables e.g.
define( 'SFTP_PATH', dirname( __FILE__ ) . '/sftp' );
Everything was working fine, however today when the project was rebuilt, I got an error in the logs, that one of our custom variables defined in wp-config-sample.php is undefined. Additionally at the start of the container there is this message:
I just discovered that there was PR merged few days ago, introducing wp-docker-config.php. I checked the repo, and https://hub.docker.com/_/wordpress/ but I couldn't find any helpful documentation on this problem. Will I still be able to use the previous setup with wp-config-sample.php, or do I need to switch to wp-docker-config.php? If so, how can I define custom variables?
The text was updated successfully, but these errors were encountered: