MUI 5 compatibility about core HOT 19 CLOSED

@oze4 Hi, I'm the author of the MUI5 conversion pull request on original repository. If this PR is any use here I could try to port it over here. Let me know if I could help you here.

from core.

I think focusing on MUI compatibility in the next repo makes more sense. As you say, there are nice features, but it is still in alpha. So I don't see a need for this to take priority right now, or in this repo.

Thoughts?

from core.

Any word on this? Would love to see the onPageChange vs onChangePage changes get pulled into this.

from core.

I just started working on a migration to v5. I started with the third level imports to fix the tests and will move to the pagination issue next, which looks like is the only error for v5.30 as far as I can see.

from core.

@VladLegkowski looks like we will have an alpha version of the library that will support mui 5.

In the meantime you can go as far as version 5.0.11 of mui, because there are no breaking changes until then

from core.

Nothing either here or in original repo. Meanwhile I bumped my PR to latest alpha, there were no immediately noticable breaking changes.

from core.

I updated upstream pull request to v5.0.0-alpha.34 level

from core.

Thank you for this. Is that the best way to handle this? Just use another branch? We currently have a next repo which we are moving to TS.

from core.

We might also start to be compatible with only MUI 5, if maintaining two versions is too much.
It's true that it is still marked as alpha, but per my use I see it as pretty stable. And there a lot of nice things with moving to version 5 😄

https://medium.com/material-ui/2020-in-review-and-beyond-55d2cab0975c

from core.

Agreed!

from core.

Hi, i noticed that react 17 is already used, but in MUI 4 only react 16 is supported officially. Was this done on purpose. Could any problems arise ?

npm WARN @material-table/[email protected] requires a peer of react@^17.0.1 but none is installed. You must install peer dependencies yourself.

from core.

@masbaehr on my projects I'm not running into any issue. I'm also using MUI 5.0.0-alpha.11 and looks like this library is compatible until that one version.

from core.

Hi guys,

I am seeing these issues with MUI5 and material-table-core/core.

Warning: Failed prop type: The prop onPageChange is marked as required in ForwardRef(TablePagination), but its value is undefined.

Warning: Unknown event handler property onChangePage. It will be ignored.

Warning: Unknown event handler property onChangeRowsPerPage. It will be ignored.

Material-UI: The fade color utility was renamed to alpha to better describe its functionality.

Package.json

    "@material-table/core": "^2.3.24",
    "@material-ui/core": "^5.0.0-alpha.23",

from core.

Ah, I already did this migration in my codebase - https://next.material-ui.com/guides/migration-v4/.

I will need to roll back a lot of stuff to make it compatible. @vmihalachi, I know I don't deserve even to ask this question, because I can only imagine how hard it is to maintain OS package like this, but you have any idea when you will have the alpha version?

Thank you.

from core.

npm ERR! Could not resolve dependency:
npm ERR! peer react@"^17.0.1" from @material-table/[email protected]

Happened again with an older project i wanted to upgrade to table/core
I think react 16 should still be supported until MUI v5 is final, as react 17 is not supported by MUI v4

from core.

@villuv is there any way to download an alpha version with these changes?

from core.

@vmihalachi Just fork my version of it from https://github.com/villuv/material-table/tree/feature/mui5 and check out feature/mui5 branch. I haven't published a built package as I haven't thorougly tested it. It builds fine with yarn install; yarn build and you can use it locally with yalc, for example. Let me know if you find anything wonky.

from core.

Feel free to send of a PR of your fork with MUI5 compatibility and we will get it merged. Thanks!

from core.

The next branch supports v5. Closing this.

from core.