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

Re-install command #19

Closed
DivineOmega opened this issue Nov 9, 2018 · 2 comments
Closed

Re-install command #19

DivineOmega opened this issue Nov 9, 2018 · 2 comments

Comments

@DivineOmega
Copy link
Contributor

Currently the install command will not update the views and config if they have changed due to a composer update. Would it be sensible to modify the lines to include the --force parameter, as follows?

$this->callSilent('vendor:publish', ['--tag' => 'wink-assets', '--force']);
$this->callSilent('vendor:publish', ['--tag' => 'wink-config', '--force']);

Alternatively, we could create a re-install command that does this. What does everyone think?

@themsaid
Copy link
Owner

I don't want to override your config file after you've changed it. To make sure the assets are updated after a change in Wink you can run:

php artisan vendor:publish --tag=wink-assets --force

@DivineOmega
Copy link
Contributor Author

@themsaid Thanks for the reply. Do you think creating a wink:reinstall or wink:upgrade command that does this for you would be useful?

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

No branches or pull requests

2 participants