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

React admin v5 #20

Closed
wants to merge 2 commits into from
Closed

React admin v5 #20

wants to merge 2 commits into from

Conversation

julienV
Copy link

@julienV julienV commented Aug 7, 2024

I updated the package.json to allow for react-admin ^5.0.0, and also add to update react query yo @tanstack/react-query because of conflicts..

Closes #19

Copy link
Collaborator

@djhi djhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR :) Unfortunately, it's not finished: some dependencies of the demo are not up to date and there are a lot of duplicates in the yarn.lock file. Besides, I think we should update the tooling as well as the demo just like we did in the react-admin repository.

@julienV
Copy link
Author

julienV commented Aug 8, 2024

Hi @djhi , ok, then i'll wait for you to take over.
I don't know or need the other packages included, i just needed to be able to upgrade react-admin to v5 in other projects, which is the case with those current modifications.

@djhi
Copy link
Collaborator

djhi commented Aug 8, 2024

Superseded by #21. Thanks!

@djhi djhi closed this Aug 8, 2024
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.

Support for react admin v5
2 participants