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

Full react 19 support #1866

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

Conversation

adubrouski
Copy link

@adubrouski adubrouski commented Jan 5, 2025

  • The existing test suites (npm test) all pass
  • For any new features or bug fixes, both positive and negative test cases have been added
  • For any new features, documentation has been added
  • For any documentation changes, the text has been proofread and is clear to both experienced users and beginners.
  • Format your code with prettier (yarn run prettier).
  • Run the Flow typechecks (yarn run typecheck).

  • Table
  • CellMeasurer
  • WindowScroller

React < 16.3.0 has been removed from peer dependencies for safe React.createRef() usage. I think this would be the right decision. We can release a new version and simply drop support for older versions of React.

ℹ️ Tests still work with findDOMNode, because react version in tests is 17. To remove findDOMNode from tests we need use @testing-library/react, but it should be in separate PR. We can also access node by ref, but this possibility will be removed in next major react release.


📦 Fork published on NPM with full react 19 support (if anyone needs it right now)

@adubrouski adubrouski force-pushed the full-react-19-support branch 6 times, most recently from f3a55d8 to e1ff44f Compare January 5, 2025 18:55
@adubrouski
Copy link
Author

Fix for #1858

@adubrouski adubrouski force-pushed the full-react-19-support branch from e1ff44f to 5bade1d Compare January 6, 2025 16:28
@Amitabh1989
Copy link

Hi can someone please okay this merge!

Long awaited feature and fixes. Just 5 mins of maintainer's time.

@adubrouski
Copy link
Author

Hi can someone please okay this merge!

Long awaited feature and fixes. Just 5 mins of maintainer's time.

Hi, you can use Fixed fork . It seems to me that this PR will not be merged into the master in the near future :)

@arunavo4
Copy link

arunavo4 commented Jan 8, 2025

@bvaughn Please merge this for React 19 support.

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

Successfully merging this pull request may close these issues.

3 participants