-
Notifications
You must be signed in to change notification settings - Fork 3
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
Migrate Mantine
to Version 7
#599
Comments
Do you have a development branch? Or how do you want help to be provided? |
Hi, I'm a contributor to this repository. I contribute by making a pull request to the main branch. |
Hey there! Yeah like Shouvik said, just fork the project and create a branch off of main. Our contribution guidelines go into a bit more detail, and our readme has instructions for setting up your local dev environment. |
I mean, that makes sense for feature adds or bugfixes. But if you want help implementing a complete rewrite in order to support Mantine 7 seems like we would want a shared branch. Someone sets the target versioning in |
Ah yeah that makes sense - I'm still pretty new to being a maintainer. I'll try and get that setup this weekend, but a lot of our styling isn't on |
Is your feature request related to a problem? Please describe.
A new major version of
Mantine
has been released.Describe the solution you'd like
There are a lot of breaking changes, but the sooner we update the less code we will need to rewrite. The biggest changes are the removal of the
createStyles
hook in favour of CSS modules, as well as some other styling changes. We will need to simultaneously update our code, allMantine
packages, andMantineDataTable
.Describe alternatives you've considered
N/A. This is required, but it can be pushed to a later milestone if needed.
Additional context
MantineDataTable
will need to be updated at the same time, but may take a few weeks to be updated as per this comment.MantineDataTable
v7 has been released.The text was updated successfully, but these errors were encountered: