Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Rename updater functions of hooks #113

Open
wants to merge 1 commit into
base: gh-pages-src
Choose a base branch
from
Open

Rename updater functions of hooks #113

wants to merge 1 commit into from

Conversation

ricguti
Copy link

@ricguti ricguti commented Sep 9, 2020

Hi 👋

Thank you for the course, it's great!

This PR renames the updater functions of hooks using the setState convention. I was reading the notes and saw that the updater functions are written in the form updateState.

On the code of SearchParams.js, the pets hook is written in the form const [pets, setPets] = useState([]);. I though maybe the other hooks could be written like that.

Rename using the setState convention for the updater functions of hooks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant