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

Fix typos in useIntervalEffect docs, update migration guide #1261

Merged
merged 1 commit into from
May 24, 2023

Conversation

JoeDuncko
Copy link
Contributor

What is the problem?

useIntervalEffect was sometimes called useInterval.

The migration guide was missing some hooks.

What changes does this PR make to fix the problem?

Renamed useInterval to useIntervalEffect everywhere.

Updated the migration guide.

@JoeDuncko JoeDuncko added the 📄 documentation Improvements or additions to documentation label May 21, 2023
@JoeDuncko JoeDuncko requested a review from ArttuOll May 21, 2023 03:03
@JoeDuncko JoeDuncko self-assigned this May 21, 2023
@codecov
Copy link

codecov bot commented May 21, 2023

Codecov Report

Merging #1261 (07e66a2) into master (10eee1c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1261   +/-   ##
=======================================
  Coverage   99.62%   99.62%           
=======================================
  Files          62       62           
  Lines        1062     1062           
  Branches      170      170           
=======================================
  Hits         1058     1058           
  Misses          2        2           
  Partials        2        2           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@JoeDuncko JoeDuncko requested a review from xobotyi May 24, 2023 12:37
@JoeDuncko JoeDuncko merged commit aeaf3e6 into master May 24, 2023
@JoeDuncko JoeDuncko deleted the fix-useinterval-naming branch May 24, 2023 13:26
@xobotyi
Copy link
Contributor

xobotyi commented May 24, 2023

🎉 This PR is included in version 23.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📄 documentation Improvements or additions to documentation released
Development

Successfully merging this pull request may close these issues.

2 participants