-
Notifications
You must be signed in to change notification settings - Fork 20
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
updated to the newest material-ui v4.12 #86
Conversation
- Fixed the tool to account for the new bugs in the docs - Ran the generator over the newer docs - Updated the links to account for the legacy status
Unfortunately @Shmew hasn't been active on here or Twitter since very early July 2021. It's possible something has happened to him :-/ |
Is there any way to use the updated version in this branch in a project that uses paket and yarn? |
Maybe we should fork it to Feliz.MUI and work on bindings for V5? |
@Linschlager: Thoughts here? I'd love @marcpiechura's suggestion, but at this point even just figuring out how to get this into a dependent project's package managers would unblock me :) |
@JustinWick . you could build the pull request branch locally and reference the dll directly, instead of the nuget package and then update the npm package separately. |
@Linschlager did you update the baseUrl? Because I think |
@JustinWick Yes, I did. (
@marcpiechura I think this would be the only sustainable option for moving forward. |
@Linschlager I think if you change this, all references will be correct: let refresh =
async {
let baseUrl = "https://v4.mui.com" //"https://material-ui.com"
... This is in the |
Not fully. |
Any plans to continue this? I'd be using v5 now if this were available! I'd help out, though I know less than nothing about modern JS frameworks, I'm a backend/embedded guy, the last time I got paid to make a GUI for a webpage was 20 years ago. |
@JustinWick I have a WIP Mui5 version on my branch, though it will probably take a while for me to finalize fixing all the special cases |
Closing this in favor of the MuiV5 "PR" |
Resolves #85