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

Added support for confirmation keyword + Setup Prettier #60

Merged
merged 3 commits into from
Oct 21, 2022

Conversation

TimMikeladze
Copy link
Contributor

@TimMikeladze TimMikeladze commented Oct 19, 2022

Closes #57

The following options have been added:

  • confirmationKeyword
  • confirmationKeywordTextFieldProps
  • getConfirmationKeywordPlaceholder

If confirmationKeyword is provided a TextField is rendered and the confirmation button is disabled. When the contents of the TextField match the value of confirmationKeyword then the confirmation button becomes enabled.

Additional stories and tests have also been added for this behavior.

Also Prettier has been configured and formatted the existing codebase. @jonatanklosko If you don't want this specific commit I can resubmit the PR or you can cherry-pick the main commit over.

Closes jonatanklosko#57

The following options have been added:

- confirmationKeyword
- confirmationKeywordTextFieldProps
- getConfirmationKeywordPlaceholder

If `confirmationKeyword` is provided a TextField is rendered and the confirmation button is disabled. When the contents of the TextField match the value of `confirmationKeyword` then the confirmation button becomes enabled.

Additional stories and tests have alse been added for this behavior.
README.md Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
.prettierignore.json Outdated Show resolved Hide resolved
@jonatanklosko
Copy link
Owner

Hey @TimMikeladze! Thanks for the PR, a couple tiny comments and we can ship it :)

@TimMikeladze
Copy link
Contributor Author

@jonatanklosko Thanks for the review. I've addressed your comments.

Copy link
Owner

@jonatanklosko jonatanklosko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TimMikeladze cool, thanks :)

@jonatanklosko jonatanklosko merged commit 17d55d4 into jonatanklosko:master Oct 21, 2022
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.

Control key word for confirm
2 participants