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

Upgrade to MudBlazor 7 #302

Open
wants to merge 5 commits into
base: blueberry
Choose a base branch
from
Open

Upgrade to MudBlazor 7 #302

wants to merge 5 commits into from

Conversation

sfmskywalker
Copy link
Member

This PR upgrades to MudBlazor from 6.x to 7.x

The latest version of MudBlzor is 7.9.0, but because MudBlazorExtensions still targets 7.3.0, we can't go all the way to 7.9.0 for now, due to an API change introduced after 7.3.0 that causes a Missing Method exception at runtime.

I created a PR for that here: CodeBeamOrg/CodeBeam.MudBlazor.Extensions#472

Refactor several components to use consistent event naming (`Value` instead of `Checked`). Enhance async methods by adding `CancellationToken` parameters. Update third-party package versions to address bugs and improvements.
Reverted MudBlazor from version 7.9.0 to 7.3.0 to resolve compatibility issues. This change ensures alignment with the current project's dependencies and avoids potential conflicts with other libraries.
Refit and Refit.HttpClientFactory packages were upgraded from version 7.1.2 to 7.2.1 in the Directory.Packages.props file. This update will ensure compatibility with newer features and bug fixes provided in the latest release.
@entech-trice
Copy link

Hey @sfmskywalker !

Thanks for the update! Anything else needed for this or is it good to go? Just need to merge now?

Thanks,
Travis

@sfmskywalker
Copy link
Member Author

Hi @entech-trice , this is already part of the main branch, so it should be all good! The purpose of this PR was to update the blueberry branch, which is an intermediary step between 3.2 and 3.3 for certain users not ready to move to the unstable 3.3 previews, while being "too big" of a change to warrant another patch on 3.2. That said, if you are you using the blueberry builds and you want the update to Blazor 7, we can certainly merge this now.

@entech-trice
Copy link

Awesome! Thanks for the clarification @sfmskywalker.

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.

2 participants