-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
More relaxed react version dependency #4530
Comments
Hi, Your ticket has been flagged as in_zendesk, it means that we recognize this as an issue worth investigating so we have moved it to our official support channel for customers (zendesk) We will carry on there with the investigation and we will update you as soon as possible. Thanks |
I'm just guessing here, but maybe it's happening because of this: https://dev.to/layershifter/how-to-kill-tree-shaking-in-webpack-with-static-properties-p72 |
Hi, We just checked and in our end it seems that is matches 16 and 17 "react": "^16.3.0 || ^17.0.0" This should allow for >= 16.3.0 and any version of 17 Is this not working on your end? |
The following doesn't match your description: Also yes, after installing |
@makinggoodsoftware did you manage to reproduce this little bug or do you need more details? |
@alippai what you linked is their devdeps, they're irrelevant to you as a user. You should check peerDeps. |
@fgblomqvist did you try it out? I'm pretty sure that if you install react 17 and ag-grid/react you'll have react-is v16 in the dependency tree and the final bundle. |
@fgblomqvist looks like it's caused by https://github.com/ag-grid/ag-grid/blob/latest/community-modules/react/package.json#L37, that depends on react-is v16: https://github.com/facebook/prop-types/blob/master/package.json#L31 |
Filed facebook/prop-types#354, but I wonder whether it makes sense to ship prop-types as a mandatory dependency at all. |
We use
and we have React 17 (in the dep tree and in our package.json). |
If you can create a package that reproduces the scenario you're describing then it can probably be looked into by the maintainers. |
I was talking about react-is. |
@fgblomqvist thanks for the insight, you are correct in your observations! Thanks |
@makinggoodsoftware this thread has every detail needed, I don't think I can do more about it. If you open the upstream issue (facebook/prop-types#354), you'll see that ag-grid won't be able to fix the issue (if it doesn't remove the prop-types dependency) |
Hello! Thank you for bringing this up. We have added this requirement to our backlog and we are tracking it with the following reference and description: We have no timeline for this at the moment, but this can still be picked up by our team when they're making modifications to this functional area of AG Grid. You can see the next release date on the product pipeline page: Now that this is recorded in our backlog, we will soon close the GitHub issue because we will not update it when this bug is fixed in a future version of AG Grid. The best way to get an update on this item is to sign up for AG Grid new release notifications. Once a new AG Grid version is released, look for the item reference above (AG-XXXX) on the changelog page to see if it's resolved in that version: Thanks again for bringing this up with us - we appreciate your time! |
Hi, In order to help us keep our issues queue clear, we are going to close this issue The official way to check the status for an issue raised here is to check its AG-xxxx ID against our pipeline and changelog …the pipeline (if not yet developed) https://www.ag-grid.com/ag-grid-pipeline/ ... or the changelog, (if the issue has been resolved) https://www.ag-grid.com/ag-grid-changelog/ We keep issues raised in the pipeline through github open for a while so that the community can add some feedback to them, and close them when there is no activity for a while. If you want to add further feedback to a closed ticket with this comment, we would recommend opening a new ticket as we don’t monitor closed issues. |
I'm submitting a ... (check one with "x")
Problem
https://github.com/ag-grid/ag-grid/blob/latest/community-modules/react/package.json contains restriction to react semver ^16 so react version 17 (and the future 18) won't match. React (and react-is) dependency will be duplicated with inconsistent versions, deduplication won't happen.
Expected behavior
No strict react 16 dependency
The text was updated successfully, but these errors were encountered: