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.

Sign up for GitHub

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

Jump to bottom

Input for only numbers? #654

Closed
championshuttler opened this issue Mar 5, 2018 · 6 comments
Closed

Input for only numbers? #654

championshuttler opened this issue Mar 5, 2018 · 6 comments

Comments

@championshuttler
Copy link

is there any input type through that only we can input number?
@SBoudrias
Copy link
Owner

There's no such prompt at the moment. The closest would be the normal input prompt with a validation checking user only inputted numbers.

I'm not against adding such a prompt to the core if someone wants to send over a PR.

CaAlden pushed a commit to CaAlden/Inquirer.js that referenced this issue Apr 5, 2018
Add Number prompt type (SBoudrias#654)
28ef015
CaAlden pushed a commit to CaAlden/Inquirer.js that referenced this issue Apr 5, 2018
Adds a NumberPrompt type for convenience when prompting for numbers(S…
58fbac2 
…Boudrias#654)
CaAlden pushed a commit to CaAlden/Inquirer.js that referenced this issue Apr 5, 2018
Adds a NumberPrompt type for convenience when prompting for numbers(S…
00ee52a 
…Boudrias#654)
SBoudrias pushed a commit that referenced this issue May 14, 2018
@CaAlden @SBoudrias
Adds a NumberPrompt type for convenience when prompting for numbers(#654
156e5b0 
) (#663)
@sjmarshy sjmarshy mentioned this issue Jun 2, 2018
Open
@karlhorky
Copy link

Can this be closed now that #663 was merged?

@dkirchhof
Copy link
Contributor

Hello, i was looking for the number prompt feature and found this issue first. You should update the documentation and typings, that this feature was merged.

@SBoudrias
Copy link
Owner

Oh good point. Anyone want to send a PR to update the README?

@SBoudrias SBoudrias reopened this Feb 19, 2019
@dkirchhof
Copy link
Contributor

Just made a pull request (#786).

@dkirchhof dkirchhof mentioned this issue Feb 21, 2019
Merged
6 tasks
@SBoudrias
Copy link
Owner

Thanks for the PR!

We can properly close this issue now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@SBoudrias @karlhorky @dkirchhof @championshuttler