Skip to content

Conversation

@rxlabz
Copy link
Contributor

@rxlabz rxlabz commented Nov 13, 2020

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

add the iOS FlutterSentryPlugin with nativeSdkIntegration and loadContexts method channels

💡 Motivation and Context

handle iOS errors and enrich payload with iOS device infos

💚 How did you test it?

added some unit tests + integration tests

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing
  • No breaking changes

🔮 Next steps

More Unit tests

@philipphofmann
Copy link
Member

Can we please fix the swift-lint errors and warnings or add a config file to ignore some of them, if we think they don't make sense. These lint issues make it super hard to take a look at the PR.

…lutter-native-ios

# Conflicts:
#	flutter/lib/sentry_flutter.dart
@rxlabz rxlabz marked this pull request as ready for review November 20, 2020 10:14
@rxlabz rxlabz requested a review from bruno-garcia as a code owner November 20, 2020 10:14
- Sentry (6.0.1):
- Sentry/Core (= 6.0.1)
- Sentry/Core (6.0.1)
- package_info (0.0.1):
Copy link
Contributor

Choose a reason for hiding this comment

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

@rxlabz this looks fishy, is this 0.0.1?

@marandaneto marandaneto merged commit c05fcbf into feat/flutter-native Nov 20, 2020
@marandaneto marandaneto deleted the feat/flutter-native-ios branch November 20, 2020 13:23
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.

4 participants