ThreadsMobileLib / edna-sdk-ios

edna iOS sdk release libraries and demo project
11 stars 4 forks source link

Не надо так. #7

Closed alexejn closed 4 years ago

alexejn commented 5 years ago

Зачем вы обернули

UNUserNotificationCenter.current().requestAuthorization(options: [.alert, .badge, .sound]) { (granted, _) in ... }

в свой метод Threads.threads().registerApplicationForRemoteNotificationsStandartOptions ?

Не надо так делать. Как теперь управлять пушами если есть другие фреймворки в приложение которые тоже конфигурируются для пушей? как задать опции [.alert, .badge, .sound] ? Оригинальное АПИ от ios должно оставаться, конфигурируете свой фреймворк через UNUserNotificationCenter, а не UNUserNotificationCenter через себя

vitkuzmenko commented 4 years ago

Вас не обязывают использовать registerApplicationForRemoteNotificationsStandartOptions. Вы можете сделать свою регистрацию. Метод сделан для удобства, можете его не использовать. Главное реализуйте методы делегата didRegisterForRemoteNotificationsWithDeviceToken и т.д. согласно примеру

Elshad commented 2 years ago

Is registerApplicationForRemoteNotificationsStandartOptions override UNUserNotificationCenter.current().requestAuthorization which called before?