Skip to content
This repository has been archived by the owner on Nov 23, 2021. It is now read-only.

WordPress Setup #59

Closed
tobeycodes opened this issue Apr 11, 2016 · 7 comments
Closed

WordPress Setup #59

tobeycodes opened this issue Apr 11, 2016 · 7 comments

Comments

@tobeycodes
Copy link
Contributor

Would you accept a pull request for a wordpress_setup flag that defaults to true.

When it is false it will not install wp-cli, download core, setup the config or install wordpress?

I always go into the var/www/ and rm -rf in the directory before I deploy my git repository anyway.

Or is there a better way to go about it?

@carlalexander
Copy link
Owner

Yep! Makes sense and I'd probably need it for #2 as well!

@lchski
Copy link
Contributor

lchski commented Apr 11, 2016

Doing some work over the weekend in a (currently private) fork, I separated the WP tasks you described (wp-cli, downloading core, etc) into its in Ansible role and playbook. I think that goes a bit against your two-command vision, @carlalexander, but it worked nicely for me.

@carlalexander
Copy link
Owner

I'm not against more roles or playbooks or commands. I just think, in this case, it's more of a setting for the WordPress role itself.

The same way you can use wordpress_dependencies: false to disable all role dependencies. You should have a way to disable the automatic installation of WordPress for setups that don't need it.

@tobeycodes
Copy link
Contributor Author

I just submitted a pull request #61

I left the task in there to create the directory and set the permissions.

Also the task for setting a cron job. I assume that people are still using WordPress

@tobeycodes
Copy link
Contributor Author

Actually, now I just realised that if we are going to deploy a Bedrock build the cron URL will be wrong

@carlalexander
Copy link
Owner

Yeah, there's already a flag for the cron too I believe.
On Mon, Apr 11, 2016 at 3:36 PM Toby [email protected] wrote:

Actually, now I just realised that if we are going to deploy a Bedrock
build the cron URL will be wrong


You are receiving this because you were mentioned.

Reply to this email directly or view it on GitHub
#59 (comment)

@carlalexander
Copy link
Owner

Just added a variable for the path to wp-cron.php.

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

No branches or pull requests

3 participants