Skip to content

Add .NET 11 breaking change doc: obsolete ModelMetadataIdentity.ForProperty overload removed - #37425

Open
Youssef1313 with Copilot wants to merge 2 commits into
mainfrom
copilot/remove-modelmetadataidentity-forproperty
Open

Add .NET 11 breaking change doc: obsolete ModelMetadataIdentity.ForProperty overload removed#37425
Youssef1313 with Copilot wants to merge 2 commits into
mainfrom
copilot/remove-modelmetadataidentity-forproperty

Conversation

Copilot AI commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

ModelMetadataIdentity.ForProperty(Type modelType, string name, Type containerType), obsolete since ASP.NET Core 3.1 (dotnet/aspnetcore#15134), was removed in dotnet/aspnetcore#67077 for .NET 11 Preview 7. This adds the corresponding breaking change article.

Changes

  • New articleaspnetcore/breaking-changes/11/modelmetadataidentity-forproperty-removed.md, following the standard breaking-change structure (version introduced, previous/new behavior, type of change, reason, recommended action, affected APIs). Classified as binary/source incompatible.
  • aspnetcore/toc.yml — TOC entry under ASP.NET Core 11 breaking changes, in alphabetical position.
  • aspnetcore/breaking-changes/11/overview.md — index table row.

Recommended migration documented in the article:

// Removed
var identity = ModelMetadataIdentity.ForProperty(
    typeof(string), "Name", typeof(Person));

// Use the PropertyInfo overload
var propertyInfo = typeof(Person).GetProperty("Name");
var identity = ModelMetadataIdentity.ForProperty(
    propertyInfo, typeof(string), typeof(Person));

… overload

Co-authored-by: Youssef1313 <31348972+Youssef1313@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove obsolete ModelMetadataIdentity.ForProperty API Add .NET 11 breaking change doc: obsolete ModelMetadataIdentity.ForProperty overload removed Aug 5, 2026
Copilot AI requested a review from Youssef1313 August 5, 2026 14:20
@Youssef1313
Youssef1313 marked this pull request as ready for review August 5, 2026 14:23
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.

[Breaking change]: ModelMetadataIdentity.ForProperty(Type modelType, string name, Type containerType) is removed

2 participants