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

Add breaking change documentation for pending model changes #4895

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

AndriySvyryd
Copy link
Member

Fixes #4891

@AndriySvyryd AndriySvyryd requested a review from a team December 3, 2024 04:40
@AndriySvyryd AndriySvyryd merged commit f7b0f83 into main Dec 3, 2024
4 checks passed
@AndriySvyryd AndriySvyryd deleted the Issue4891 branch December 3, 2024 20:38
- **Mitigation**: Add a new migration, examine its contents to locate the cause, and replace the dynamic data with a static, hardcoded value in the model. The migration should be recreated after the model is fixed. If dynamic data has to be used for seeding consider using [the new seeding pattern](/ef/core/what-is-new/ef-core-9.0/whatsnew#improved-data-seeding) instead of `HasData()`.
- The last migration was created for a different provider than the one used to apply the migrations.
- **Mitigation**: This is an unsupported scenario. The warning can be suppressed using the code snippet below, but this scenario will likely stop working in a future EF Core release. The recommended solution is [to generate a separate set of migrations for each provider](xref:core/managing-schemas/migrations/providers).
- The migrations are generated or choosen dynamically by replacing some of the EF services.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- The migrations are generated or choosen dynamically by replacing some of the EF services.
- The migrations are generated or chosen dynamically by replacing some of the EF services.

Copy link
Member Author

Choose a reason for hiding this comment

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

Feel free to massage this text further to make it useful to more of our userbase

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.

Document Pending Model Changes warning
3 participants