Skip to content

Update recycling.md #1392

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update recycling.md #1392

wants to merge 1 commit into from

Conversation

valtism
Copy link

@valtism valtism commented Oct 22, 2024

Description

With the beta release of the react-compiler eslint plugin, accessing a refs .current during render is marked as an error.

This is also warned against in the official docs

Instead, we can wrap this in a useEffect which is the recommended practice.

I have also updated some of the language to avoid long sentences and removed the reference to the outdated docs.

With the beta release of the `react-compiler` [eslint plugin](https://www.npmjs.com/package/eslint-plugin-react-compiler), accessing a refs `.current` during render is marked as an error.

This is also warned against in the [official docs](https://react.dev/reference/react/useRef)

Instead, we can wrap this in a `useEffect` which is the recommended practice.

I have also updated some of the language to avoid long sentences and removed the reference to the outdated docs.
@valtism
Copy link
Author

valtism commented Oct 22, 2024

I have signed the CLA!

# 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