Skip to content

Conversation

@jkasten2
Copy link
Member

@jkasten2 jkasten2 commented Nov 28, 2016

  • Fixed local notification selectors not firing when app is in focus
    • userNotificationCenter:willPresentNotification:withCompletionHandler was not calling legacy selectors as it should.
  • Fixed issue where local notifications created with presentLocalNotificationNow would not fire their AppDelegate selector.
  • Removed deprecated OSUserNotificationCenterDelegate class
    • Due to very low usage and complexly to maintain, it’s being removed before a major release version.

This change is Reviewable

* Fixed local notification selectors not firing when app is in focus
   - userNotificationCenter:willPresentNotification:withCompletionHandler was not calling legacy selectors as it should.
* Fixed issue where local notifications created with presentLocalNotificationNow would not fire their AppDelegate selector.
* Removed deprecated OSUserNotificationCenterDelegate class
   - Due to very low usage and complexly to maintain, it’s being removed before a major release version.
@jkasten2 jkasten2 merged commit 1d53c50 into master Nov 28, 2016
@jkasten2 jkasten2 deleted the infocus_swizzling_fixes branch November 29, 2016 04:49
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.

2 participants