Skip to content
This repository has been archived by the owner on Feb 2, 2023. It is now read-only.

Use setState updater to access state #32

Merged
merged 2 commits into from
Sep 23, 2018
Merged

Use setState updater to access state #32

merged 2 commits into from
Sep 23, 2018

Conversation

morganfree
Copy link
Collaborator

@morganfree morganfree requested review from GoldenTK and stpoa and removed request for GoldenTK September 15, 2018 18:52
GoldenTK
GoldenTK approved these changes Sep 21, 2018
View reviewed changes
Copy link
Collaborator

@GoldenTK GoldenTK left a comment

Choose a reason for hiding this comment

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

Small change requested.

client/src/generic/SelectTags.tsx Show resolved Hide resolved
@morganfree morganfree merged commit 9899082 into master Sep 23, 2018
@morganfree morganfree deleted the fix/no-access-state-props-in-set-state branch September 23, 2018 14:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@GoldenTK GoldenTK GoldenTK approved these changes

@stpoa stpoa Awaiting requested review from stpoa

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@morganfree @GoldenTK