Skip to content

Change default embedded discriminator property from "Discriminator" to "$type"#34340

Merged
ajcvickers merged 2 commits intomainfrom
ReadMe
Aug 6, 2024
Merged

Change default embedded discriminator property from "Discriminator" to "$type"#34340
ajcvickers merged 2 commits intomainfrom
ReadMe

Conversation

@ajcvickers
Copy link
Copy Markdown
Contributor

Fixes #34269

Use the following to revert back to the old behavior:

modelBuilder.HasEmbeddedDiscriminatorName("Discriminator");

@ajcvickers ajcvickers requested a review from a team August 2, 2024 11:47
@ajcvickers ajcvickers force-pushed the ReadMe branch 2 times, most recently from b47cb79 to d267a60 Compare August 2, 2024 16:58
Comment thread src/EFCore/Metadata/Conventions/IModelEmbeddedDiscriminatorNameConvention.cs Outdated
…o "$type"

Fixes #34269

Use the following to revert back to the old behavior:

```C#
modelBuilder.HasEmbeddedDiscriminatorName("Discriminator");
```
@ajcvickers ajcvickers merged commit c3963d6 into main Aug 6, 2024
@ajcvickers ajcvickers deleted the ReadMe branch August 6, 2024 12:44
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.

Cosmos: Switch the default discriminator property to $type

2 participants