Skip to content

[release/8.0-preview6] [Blazor] Form mapping error handling and validation integration for SSR Blazor #49031

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

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 26, 2023

Backport of #48990 to release/8.0-preview6

/cc @javiercn

[Blazor] Form mapping error handling and validation integration for SSR Blazor

Adds support for error handling and validation to Server Side Rendered Blazor.

Description

Integrates the form data binding experience with Blazor and provides support for error handling.

Fixes #46983 (in this specific format)

Customer Impact

This is one of the remaining major functionalities for Blazor Server Side rendering, so we want to make sure we can get as much feedback as possible on the experience. Specially since we are planning an app building exercise.

Regression?

  • Yes
  • No

[If yes, specify the version the behavior has regressed from]

Risk

  • High
  • Medium
  • Low

The changes should be additive, and we've included a significant amount of test coverage for the new functionality.

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

When servicing release/2.1

  • Make necessary changes in eng/PatchConfig.props

@ghost ghost added the area-blazor Includes: Blazor, Razor Components label Jun 26, 2023
Copy link
Member

@SteveSandersonMS SteveSandersonMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already approved equivalent PR against main

@captainsafia
Copy link
Member

Already approved equivalent PR against main

I've merged to both this branch and the one set to target main a change that updates the minimal APIs implementation to react to the API changes in this PR. This should unblock the test failures in the build.

@javiercn javiercn force-pushed the backport/pr-48990-to-release/8.0-preview6 branch from 7316aa1 to 7038abc Compare June 26, 2023 18:29
@mkArtakMSFT mkArtakMSFT added the Servicing-consider Shiproom approval is required for the issue label Jun 26, 2023
@ghost
Copy link

ghost commented Jun 26, 2023

Hi @github-actions[bot]. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@mkArtakMSFT mkArtakMSFT added Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Jun 26, 2023
@ghost
Copy link

ghost commented Jun 26, 2023

Hi @github-actions[bot]. This PR was just approved to be included in the upcoming servicing release. Somebody from the @dotnet/aspnet-build team will get it merged when the branches are open. Until then, please make sure all the CI checks pass and the PR is reviewed.

@mkArtakMSFT mkArtakMSFT merged commit 417ce3b into release/8.0-preview6 Jun 26, 2023
@mkArtakMSFT mkArtakMSFT deleted the backport/pr-48990-to-release/8.0-preview6 branch June 26, 2023 20:01
@ghost ghost added this to the 8.0-preview6 milestone Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components Servicing-approved Shiproom has approved the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants