Skip to content
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

Added PUBLIC_DIR argument to customise web directory #5

Open
wants to merge 1 commit into
base: 1.x
Choose a base branch
from

Conversation

Patabugen
Copy link

Allows users to change the public-directory name from web to anything they like by (optionally) adding a PUBLIC_PATH argument to the docker-compose file.

I've applied the change to 7.4, 8.0 and 8.1 runtimes and it should be backwards compatible without a problem.

If users have published the Dockerfile or supervisord.conf they'll need to update both if they want to use the argument.

Resolves: #4

Adds these docs to the readme:

Alternative Public Path

If your WordPress files are in a web/ directory in your repository then you're good to go - otherwise set the PUBLIC_DIR arg in docker-compose.yaml.

version: '3'
services:
    wordpress.test:
        build:
            context: ./vendor/sterner-stuff/wordpress-sail/runtimes/8.1
            dockerfile: Dockerfile
            args:
                WWWGROUP: '${WWWGROUP}'
                PUBLIC_DIR: 'public' <-- path to the directory containing wp-config.php relative to your docker-compose.yaml file
        # ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the docroot customisable (to something other than 'web')
1 participant