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

Step 1 - Convert Filter component #5

Open
wants to merge 6 commits into
base: 00-base
Choose a base branch
from
Open

Step 1 - Convert Filter component #5

wants to merge 6 commits into from

Conversation

danieltott
Copy link
Owner

@danieltott danieltott commented May 30, 2019

Here we use useState for managing the state of the form inputs, as well as keeping track of whether the form is valid.

We also converted the validate function to async, to see how that interaction affected our component. This lead to adding an effect via useEffect that runs our validation for us, as well as the ability to check whether our validation task is still current via isCurrent.

@danieltott danieltott changed the title Step One - Convert Filter component Step 1 - Convert Filter component May 30, 2019
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