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

🔥 Remove deprecated hooks and co #503

Merged
merged 6 commits into from
Feb 27, 2024
Merged

Conversation

juliencrn
Copy link
Owner

Use the major release to batch all breaking changes in the hooks.

This PR removes previously deprecated hooks and old hooks' signatures.

Removed

  • useDebounce: Replaced by both useDebouceValue and useDebounceCallback.
  • useEffectOnce: unnecessary abstraction.
  • useElementSize: Replaced by more performant useResizeObserver.
  • useFetch: Prefer RSC, framework methods, React's use, React-query, useSWR...
  • useImageOnLoad: Too opinionated.
  • useIsFirstRender: Not comply with the React functional mindset (cf: React 18 render twice in strict mode).
  • useLockedBody: Replaced by the more generic useScrollLock.
  • useSsr: It was not a React hook, other hooks can do the job.
  • useUpdateEffect: unnecessary abstraction.

Hooks with dropped legacy signature

  • useCountDown
  • useDarkMode
  • useIntersectionObserver
  • useMediaQuery
  • useTernaryDarkMode

Additional

Also updated the website to allow hook pages without code or demo. Later, we could consider removing the doc for removed hooks and use permanent redirection to the alternative.

@juliencrn juliencrn added the V3 Something introduces breaking changes. label Feb 22, 2024
Copy link

changeset-bot bot commented Feb 22, 2024

🦋 Changeset detected

Latest commit: edd3814

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
usehooks-ts Major
www Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@BlankParticle
Copy link
Contributor

Before v3 remember to merge the automated workflow PR.

@juliencrn juliencrn changed the title Remove deprecated hooks and co 🔥 Remove deprecated hooks and co Feb 22, 2024
@juliencrn juliencrn merged commit eeacd8c into v3 Feb 27, 2024
2 checks passed
@juliencrn juliencrn deleted the remove-deprecated-hooks-and-co branch February 27, 2024 18:37
juliencrn added a commit that referenced this pull request Feb 27, 2024
* Remove @deprecated hooks' function overload keeping the latest signature only

* Remove @deprecated hooks only keeping updated version of their .md doc files

* Remove deprecated hook exclusions in scripts + little clean up

* website: allow hook without code nor demo wheneven there is a .md file

* 🔖 Add changeset
juliencrn added a commit that referenced this pull request Mar 4, 2024
* Remove @deprecated hooks' function overload keeping the latest signature only

* Remove @deprecated hooks only keeping updated version of their .md doc files

* Remove deprecated hook exclusions in scripts + little clean up

* website: allow hook without code nor demo wheneven there is a .md file

* 🔖 Add changeset
juliencrn added a commit that referenced this pull request Mar 5, 2024
* Remove @deprecated hooks' function overload keeping the latest signature only

* Remove @deprecated hooks only keeping updated version of their .md doc files

* Remove deprecated hook exclusions in scripts + little clean up

* website: allow hook without code nor demo wheneven there is a .md file

* 🔖 Add changeset
juliencrn added a commit that referenced this pull request Mar 8, 2024
* 🏗️ Move the full workspace to ES Module (#502)

* 🔥 Remove deprecated hooks and co (#503)

* ♻️ Prefer type over interface (#515)

* ♻️ Generate documentation from JSDoc using typedoc (#519)

* 📝 Fix documentation build with some little improvements

* 👷 Add "DOM.Iterable" as the code runs in the DOM

* 📝 Add Algolia doc search command menu (#513)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V3 Something introduces breaking changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants