-
Notifications
You must be signed in to change notification settings - Fork 497
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
Release kMXKAccountAPNSActivityDidChangeNotification observer when de… #7697
Conversation
…allocation controller Signed-off-by: Nicolas Buquet <[email protected]>
f4e4cd0
to
6b8add7
Compare
…vers Signed-off-by: Nicolas Buquet <[email protected]>
3d09ef1
to
af0ea7a
Compare
…d in `destroy`. Signed-off-by: Nicolas Buquet <[email protected]>
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!
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #7697 +/- ##
===========================================
+ Coverage 11.86% 12.37% +0.50%
===========================================
Files 1650 1650
Lines 163863 163869 +6
Branches 67219 67267 +48
===========================================
+ Hits 19446 20277 +831
+ Misses 143777 142927 -850
- Partials 640 665 +25
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Remove registered observer
kMXKAccountAPNSActivityDidChangeNotification
in dealloc of SettingsViewController.Pull Request Checklist