-
-
Notifications
You must be signed in to change notification settings - Fork 239
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
Feat: Use Hint
for screenshots
#1165
Feat: Use Hint
for screenshots
#1165
Conversation
|
Codecov ReportBase: 76.27% // Head: 65.59% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## feat/introduce-hint-data-bag #1165 +/- ##
=================================================================
- Coverage 76.27% 65.59% -10.69%
=================================================================
Files 11 4 -7
Lines 333 186 -147
=================================================================
- Hits 254 122 -132
+ Misses 79 64 -15 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@@ -4,7 +4,8 @@ | |||
|
|||
### Features | |||
|
|||
- Feat: Introduce Hint data bag ([#1136](https://github.com/getsentry/sentry-dart/pull/1136)) | |||
- Feat: Introduce `Hint` data bag ([#1136](https://github.com/getsentry/sentry-dart/pull/1136)) | |||
- Feat: Use `Hint` for screenshots ([#1165](https://github.com/getsentry/sentry-dart/pull/1165)) |
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.
- 🚫 The changelog entry seems to be part of an already released section
## 7.0.0
.
Consider moving the entry to the## Unreleased
section, please.
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.
Left a last comment, otherwise LGTM.
…-screenshot # Conflicts: # dart/lib/src/hint.dart
📜 Description
Attaches screenshots to
Hint
💡 Motivation and Context
Users can now filter out screenshots with an event processor or through before send. Relates to #1127
💚 How did you test it?
Added (migrated) unit tests.
📝 Checklist