aaemnnosttv / wp-cli-valet-command

Harness the power of Laravel Valet for creating fully functional WordPress installs in seconds.
https://aaemnnost.tv/wp-cli-commands/valet/
299 stars 31 forks source link

Timezone Setting #65

Closed neilgee closed 4 years ago

neilgee commented 4 years ago

I was hoping that --timezone_string="Sydney" would work by adding it to a creation script, but it didn't - is it possible to add it?

wp valet new $1 --version="5.3.2" --locale="en_AU" --timezone_string="Sydney" --dbname="wp_$1" --dbuser="root" --dbpass="" --dbprefix="wp_" --admin_user="admin" --admin_password='password' --admin_email="support@support.com.au" --unsecure

aaemnnosttv commented 4 years ago

The command mostly interacts with the underlying core commands for config create and core install, neither of which currently offer an option for the timezone.

You might propose this as a new option (probably to core install) to the core command. If core supports it, I'd be happy to add support for it here 😄

neilgee commented 4 years ago

ok thanks will raise an issue there

neilgee commented 4 years ago

They are asking what the core command is, have I worded that Incorrect?https://github.com/laravel/valet/issues/948#issuecomment-624696046

aaemnnosttv commented 4 years ago

Oh sorry, I meant core as in wp-cli which is what would handle anything WP-related like this.

This is the command I would think is the most relevant to add it to: https://github.com/wp-cli/core-command#wp-core-install

neilgee commented 4 years ago

Ah ok, wasn't thinking straight - I just added it to my shell script on site creation and it sets the value, works fine. wp option update timezone_string "Australia/Sydney"