-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add clarifying example in migration guide to v10 #3672
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
Conversation
Added a concrete example on how to check for concrete types in order to mutate model properties in the migration guide. Without the example, I personally read over the bullet point in the migration guide. It might or might not be helpful for others, if you don't think so please feel free to ignore the PR.
martincostello
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, just a few copy-edits.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3672 +/- ##
=======================================
Coverage 94.53% 94.53%
=======================================
Files 111 111
Lines 3861 3861
Branches 778 778
=======================================
Hits 3650 3650
Misses 211 211
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Added a concrete example on how to check for concrete types in order to mutate model properties in the migration guide. Without the example, I personally read over the bullet point in the migration guide.
It might or might not be helpful for others, if you don't think so please feel free to ignore the PR.