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

Is there a plan of migration to material-ui v5? #1513

Open
yaminyaylo opened this issue Sep 11, 2020 · 23 comments
Open

Is there a plan of migration to material-ui v5? #1513

yaminyaylo opened this issue Sep 11, 2020 · 23 comments

Comments

@yaminyaylo
Copy link

yaminyaylo commented Sep 11, 2020

The issue is when I updated the Material-UI library to the latest version I found out that MUI-datatables doesn't support the Material-UI library v5.

Updated Material-UI libs to the following versions:

  • "@material-ui/core": "5.0.0-alpha.9",
  • "@material-ui/icons": "5.0.0-alpha.7",
  • "@material-ui/lab": "5.0.0-alpha.9",
  • "@material-ui/pickers": "^4.0.0-alpha.12",
  • "@material-ui/styles": "5.0.0-alpha.8",

Expected Behavior

No errors and crashed on the Table filter action.

Current Behavior

There are errors and crashed on the Table filter action press.

I guess this issue is due to Material-UI library new version - The onE* transition props were removed. Use TransitionProps instead. https://github.com/mui-org/material-ui/releases/tag/v5.0.0-alpha.7
Screenshot 2020-09-11 at 17 06 58
Screenshot 2020-09-11 at 17 07 20
Screenshot 2020-09-11 at 17 07 20

Steps to Reproduce (for bugs)

  1. update material-ui and it's dependant libs to the latest
  2. fix all breaking changes in your app
  3. go to the page you have a table based on MUI-datatables
  4. suppose it has passed data inside the table and filter option is true
  5. click on the Filter icon button

Your Environment

Tech Version
Material-UI 5.0.0-alpha.9
MUI-datatables 3.4.1
React 16.13.1
browser Chrome (latest)
etc
@patorjk
Copy link
Collaborator

patorjk commented Sep 11, 2020

I would accept pull requests to get the table working on version 5. If it requires breaking changes, it'd require a version bump and would probably be worth waiting until the official version 5 is released. I have not yet used version 5.

Also worth mentioning: The Material UI team is actually going to release an official data-table component for version 5.0. I talked with the lead Material UI dev a few months ago about it, and chatted back and forth through email around 6-7 weeks ago with the developer they have working on the official data-table (I tried the table out and then made a bunch of suggestions - the table looks pretty good and the developer seemed very receptive to my feedback). The goal is to get rid of the need for 3rd party data-tables like mui-datatables and material-table. However, the unfortunate thing is that many of the features that mui-datatables currently supports will be locked behind a paid enterprise version (at least that was the case 6-7 weeks ago), so I'm assuming there will still a place for mui-datatables and material-table, but I've sort of taken a wait-and-see approach by not doing too much development, as I'm not sure what direction the community will go. However, I'm open to PRs from other devs.

@yaminyaylo
Copy link
Author

yaminyaylo commented Sep 16, 2020

@patorjk thanks a lot for a very quick response and explanation.

I will think about what to do next.

@oliviertassinari
Copy link

oliviertassinari commented Mar 10, 2021

To add to @patorjk's message, it would be great that the library upgrades to @material-ui/core v5. From a high-level point of view, we are pushing in the following directions:

  • Material-UI core, which is meant to empower design systems. Completly MIT. We are currently expanding by polishing the core to the extreme, a11y, bugs, features, customization, documentation. We currently have a default Material Design theme and soon we will provide a second theme, more aligned with the latest design trends. We monetize with the designers, through paid design kits (Figma, Adobe XD, Sketch) and probably more tools in the future, advertising and premium templates.
  • Material-UI X, which is meant to provide advanced components, combo box, date picker, tree view, data grid, scheduler, charts, etc. We can't sustain all this effort without having some part of it paid. We employs three different tradeoffs there:
    • MIT. For features that the community can sustain. It's for when there are enough contributions and the problem are simple enough. This work for instance, for the date picker (a bit borderline in terms of complexity, but we also have to give back).
    • Pro (inexpensive). For when there might be open-source alternatives but nowhere need the level of polish we aim for. For instance, the xgrid might have paid features that https://github.com/adazzle/react-data-grid have free. Having alternatives like https://github.com/gregnb/mui-datatables or https://github.com/mbrn/material-table is actually healthy for the ecosystem.
    • Premium. For what you would usually get in the most advanced premium UI libraries, ag-Grid, highcharts, Kendo-UI. We don't expect any open-source solutions to come near before a decade.

@patorjk
Copy link
Collaborator

patorjk commented Mar 10, 2021

@wdh2100 has taken over as the main maintainer. I wouldn't mind helping out though if he needs some assistance with this.

@davidcraddock
Copy link

so will this project be updated to v5?

@wdh2100
Copy link
Collaborator

wdh2100 commented Mar 11, 2021

@davidcraddock

Yes, I hava plan to proceed.
When material-ui enter the stable phase, I will switch to v5.

https://github.com/gregnb/mui-datatables/issues?q=is%3Aissue+label%3A%22material+ui+5.0%22+

@davidcraddock
Copy link

@wdh2100 thanks for the clarification

@yaminyaylo
Copy link
Author

@wdh2100 that's great news, looking forward to material-UI v.5 entering the stable phase

@artivilla
Copy link

Given material-ui release v5.0.0-beta.0 now encourages new projects to start with v5 I think it would be beneficial to support v5. Thoughts?

@narkai
Copy link

narkai commented Jul 23, 2021

We really need this, because of all the changes in Material it does not make sense to start a project with the V4 anymore...

@narkai
Copy link

narkai commented Aug 5, 2021

After days of struggling with Material v5 Datagrid component, I rolled back to Material v4 & mui-datatables.

Comparing the two libraries really showed how mui-datatables is relevant, it's just so good !

Even though Datagrid seems to go in the right direction, IMO it's still far behind in term of capabilities & developer experience (especially custom filtering / sorting).

@yaminyaylo
Copy link
Author

yaminyaylo commented Aug 5, 2021

yeah, all our codebase is waiting for migration of mui-datatables to material-ui v5 beta (actually we now using "5.0.0-alpha.5"), but mui-datatables is broken after increasing the version of the material-ui core higher than "5.0.0-alpha.5" :(

@davidcraddock
Copy link

Yeah I had to update my forked repository above to use the latest beta.2 material-ui today until there’s an official release

@MendelBak
Copy link

@yaminyaylo Is this library still maintained? I noticed there hasn't been many PRs or issues closed in the past few months. Is this because you're waiting till v5 of MUI comes out, and to migrate mui-datables to that, before allowing changes?

@yaminyaylo
Copy link
Author

yaminyaylo commented Aug 17, 2021

@yaminyaylo Is this library still maintained? I noticed there hasn't been many PRs or issues closed in the past few months. Is this because you're waiting till v5 of MUI comes out, and to migrate mui-datables to that, before allowing changes?

@MendelBak I hope it is still maintained because it will be a sad story :)
I'm waiting that mui-datatables devs will use material-ui v5 inside mui-datatables.

@wdh2100
Copy link
Collaborator

wdh2100 commented Aug 18, 2021

@wdh2100 thanks for the clarification

@davidcraddock @lapico @yaminyaylo @MendelBak

I'll try to make an beta version

The target version is @material-ui/core@5.0.0-beta.4

Given material-ui release v5.0.0-beta.0 now encourages new projects to start with v5 I think it would be beneficial to support v5. Thoughts?

@artivilla

thanks!

@luluhoc
Copy link
Contributor

luluhoc commented Aug 27, 2021

Yeah I updated my app to v5 material just to find issue with mui-datatables :( please update it.

@gnowland
Copy link

@wdh2100 looks like v5 stable is two weeks away, will this be ready?

@zxhmike
Copy link
Contributor

zxhmike commented Sep 1, 2021

Hi @wdh2100 ,

Our team spent some time upgrading mui-datatables to Material-UI 5. The preliminary PR can be found here: #1772

Sorry, I just saw the updates on this issue that you were making some effort to do a similar upgrade. Hopefully, we did not step on your toes.

Please review the PR and give us feedback. We are able to bring up the examples page, but there are some known issues that we are looking into, which can be found in the description of the PR.

Thank you! Looking forward to your reply.

@yaminyaylo
Copy link
Author

yaminyaylo commented Sep 20, 2021

@davidcraddock

Yes, I hava plan to proceed.
When material-ui enter the stable phase, I will switch to v5.

https://github.com/gregnb/mui-datatables/issues?q=is%3Aissue+label%3A%22material+ui+5.0%22+

https://github.com/mui-org/material-ui/releases/tag/v5.0.0
material-ui recently has released v5, do you think we can expect mui-datatables to work on migrating to material-ui core latest?

@293nav
Copy link

293nav commented Oct 1, 2021

Any update on upgrading this library to support mui v5. Material-ui has released a new stable version. I noticed that in the stable version the imports are now made from @mui/material instead of @material-ui/

@3BDO-debug
Copy link

I got it working with material ui v5 successfully by fixing some dependencies and little tweaks check it out now.

https://github.com/3BDO-debug/MUIDataTable-MUI-V5

@yaminyaylo
Copy link
Author

Huge thanks to the team who migrated to material UI v5! You ROCK!!!

Can someone close the issue?

CC @patorjk @wdh2100

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

No branches or pull requests