Skip to content

Allow starter kit installer interactivity#56

Merged
jesseleite merged 3 commits intomasterfrom
allow-starter-kit-interactivity
Oct 6, 2022
Merged

Allow starter kit installer interactivity#56
jesseleite merged 3 commits intomasterfrom
allow-starter-kit-interactivity

Conversation

@jesseleite
Copy link
Contributor

@jesseleite jesseleite commented Sep 30, 2022

After releasing statamic/cms#6792, we noticed that starter-kit:install is run non-interactively, suppressing any user input prompts in a starter kit's post-install hook. This PR fixes this.

We're also removing the 'temporary' comment on --cli-install, because the starter kit installer should never handle the creation of a super user when using this cli installer. It needs to be here anyway, for when installing statamic/statamic without a starter kit.

Note: The above-mentioned post-install hook interactivity doesn't work in Windows due to the lack of TTY support in Windows. For these situations, we instead output any cached post-install instructions passed along by statamic/cms...

CleanShot 2022-10-04 at 17 48 46

@jesseleite jesseleite marked this pull request as ready for review October 6, 2022 00:45
@jesseleite jesseleite merged commit d51b006 into master Oct 6, 2022
@jesseleite jesseleite deleted the allow-starter-kit-interactivity branch October 6, 2022 00:45
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.

1 participant