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

Add support for wordpress network-mode #67

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

graphiclunarkid
Copy link
Contributor

Wordpress network-mode allows multiple separate blogs to be served from
the same instance of the software. These can be served as separate
subdirectories or separate subdomains. With the latter option, a plugin
is included to allow separate apex domains to be mapped to each
subdomain.

Wordpress.com is the largest example of wordpress running in network
mode.

This branch also:

  • Makes configurable the domain and subdirectory from which wordpress is served
  • Refactors the wordpress role
  • Includes defaults for all optional parameters
  • Includes handlers to ensure these are run when the role is executed in
    isolation
  • Updates the role documentation to explain the new options
  • Updates wordpress to v4.4.2

Resolves #66

Wordpress network-mode allows multiple separate blogs to be served from
the same instance of the software. These can be served as separate
subdirectories or separate subdomains. With the latter option, a plugin
is included to allow separate apex domains to be mapped to each
subdomain.

Wordpress.com is the largest example of wordpress running in network
mode.

This branch also:

- Makes configurable the domain and subdirectory from which wordpress is served
- Refactors the wordpress role
- Includes defaults for all optional parameters
- Includes handlers to ensure these are run when the role is executed in
  isolation
- Updates the role documentation to explain the new options
- Updates wordpress to v4.4.2

Resolves #66
Copy link
Contributor

@kheops2713 kheops2713 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • nginx vhost file is redundant with nignx role vhost file ("websites" variable)
  • other actions are redundant with "websites" variable, like creating domain directories
  • add meta/main.yml file to autoinclude nginx and php-fpm role
  • try re-adding "alias" nginx directive for wordpress subdomain configuration where location is '/'
  • make sure PHP is called for all domains serving a wordpress site, in case of a multisite with several domain names, by copying the nginx included configuration location / into every domain's configuration path
  • remove handlers redundant with nginx role

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

Successfully merging this pull request may close these issues.

2 participants