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

[5.x] Added default value on selection question while creating new application #300

Conversation

bigship-prashant
Copy link
Contributor

This pull request aims to enhance the Laravel installer by introducing default values for selection questions during the process of creating a new application. The addition of default values streamlines the installation process, making it more user-friendly and efficient.

With this update, I've implemented default values for certain selection questions that users encounter while creating a new Laravel application. These defaults are thoughtfully chosen to align with best practices and common preferences, ensuring a smoother on-boarding experience for both new and experienced users.

By providing default values, users can simply accept the pre-filled options if they align with their requirements. This significantly reduces the number of manual inputs, making the installation process faster and more straightforward.

@driesvints
Copy link
Member

Aren't values already selected by default? Afaik this option is only to select specific ones. I can always just hit enter and it uses the one that's currently selected.

@driesvints driesvints changed the title 5.x: Added default value on selection question while creating new application [5.x] Added default value on selection question while creating new application Jan 5, 2024
@bigship-prashant
Copy link
Contributor Author

Aren't values already selected by default? Afaik this option is only to select specific ones. I can always just hit enter and it uses the one that's currently selected.

Nope, I cannot use it on my windows 11 pc. If you use Mac or Linux, I cannot tell but in windows we need to type. So this PR is for reducing that

@driesvints driesvints requested a review from jessarcher January 5, 2024 10:44
@taylorotwell taylorotwell merged commit 7e97a1d into laravel:5.x Jan 6, 2024
8 checks passed
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.

3 participants