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

Rename Hydrate to HydrationBoundary #4692

Closed
TkDodo opened this issue Dec 23, 2022 · 8 comments
Closed

Rename Hydrate to HydrationBoundary #4692

TkDodo opened this issue Dec 23, 2022 · 8 comments
Assignees
Labels
Milestone

Comments

@TkDodo
Copy link
Collaborator

TkDodo commented Dec 23, 2022

and remove useHydrate hook

@TkDodo TkDodo added the v5 label Dec 23, 2022
@TkDodo TkDodo added this to the v5 milestone Dec 23, 2022
@ghost
Copy link

ghost commented Dec 24, 2022

Should the useHydrate hook be replaced with anything or just completely removed?

@TkDodo
Copy link
Collaborator Author

TkDodo commented Dec 24, 2022

Should the useHydrate hook be replaced with anything or just completely removed?

It can be removed

@prateek3255
Copy link
Contributor

Hey @TkDodo I would like to contribute to this issue and write the codemod for it

@TkDodo
Copy link
Collaborator Author

TkDodo commented Jan 12, 2023

perfect, it's yours

@TkDodo
Copy link
Collaborator Author

TkDodo commented Jan 17, 2023

@prateek3255 please wait with the code-mod until we have the setup in place, see:

we'll probably create separate code-mod issues then for each one.

@sshmaxime
Copy link

Hey, I think you should update the documentation here: <Hydrate state={dehydratedState}>

@TkDodo
Copy link
Collaborator Author

TkDodo commented Jan 19, 2024

you're looking at v4 docs, the rename happened in v5

@sshmaxime
Copy link

My bad, thanks dude 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants