-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
The user in Dart/Flutter was renamed to SentryUser #3244
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
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sentry/sentry-docs/diqJPPTx32iG9ZX2tysKFMCUNPwB |
|
should merge only once v5 is GA |
|
we'd need to do the migration page from v4 to v5 also on our docs, it could be something similar to what we've done for the changelog. See Android docs: https://docs.sentry.io/platforms/android/migration/#migrating-from-iosentrysentry-410-to-iosentrysentry-420 |
marandaneto
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.
LGTM
lets wait for v5 GA
|
I've made it a draft to not accidentally merge it, its good to go though |
Co-authored-by: Manoel Aranda Neto <5731772+marandaneto@users.noreply.github.com>
See getsentry/sentry-dart#357 and #3196