Skip to content

Commit

Permalink
[flutter_local_notifications] update info on permissions request feat…
Browse files Browse the repository at this point in the history
…ure and firebase_messaging plugin compatibility issues (MaikuB#491)

* update description on request notification permissions feature in the readme

* bump plugin version and add details on using forked firebase_messaging plugin
  • Loading branch information
MaikuB authored Feb 22, 2020
1 parent 748071f commit ac6563c
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 6 deletions.
9 changes: 7 additions & 2 deletions flutter_local_notifications/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
# [1.1.7+1]

* Minor update to readme on description around requesting notification permissions
* Add link to forked `firebase_messaging` plugin to readme for those that want to use it whilst the PR to fix the compatibility issues with this plugin is waiting to be reviewed

# [1.1.7]

* [iOS] Added `requestPermissions()` method to `IOSFlutterLocalNotificationsPlugin` class. This can be used to request notification permissions separately from plugin initialisation. To facilitate this the `IOSFlutterLocalNotificationsPlugin` and `AndroidFlutterLocalNotificationsPlugin` now expose a static `instance` property that can be used obtain the platform-specific implementation of the plugin so that platform-specific methods can be used. Thanks to the PR from [Dariusz Łuksza](https://github.com/dluksza)
* Updated documentation to clarify that `getNotificationAppLaunchDetails()` is intended to be used more on if a notification from this plugin triggered launch an application
* Updated API docs for consistency and to better follow the guidelines on effective Dart documentation.
* Updated API docs for consistency and to better follow the guidelines on effective Dart documentation

# [1.1.6]

Expand All @@ -11,7 +16,7 @@

# [1.1.5+1]

* No functional changes. Fixed a reported formatting issue.
* No functional changes. Fixed a reported formatting issue
* Mention removal of named constructor argument in 1.1.0 changelog entry
* Add API docs to `FlutterLocalNotificationsPlugin.private()` on how it could be used for testing
* Update notes on testing to mention that the `FlutterLocalNotificationsPlugin.private()` named constructor may be of use
Expand Down
5 changes: 2 additions & 3 deletions flutter_local_notifications/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,7 @@ A cross platform plugin for displaying local notifications.
* [Android] Group notifications
* [Android] Show progress notifications
* [Android] Configure notification visibility on the lockscreen
* [iOS] Request notification permissions when needed (e.g. in response to user turning on a setting)
* [iOS] Customise the permissions to be requested around displaying notifications
* [iOS] Request notification permissions and customise the permissions being requested around displaying notifications

Note that this plugin aims to provide abstractions for all platforms as opposed to having methods that only work on specific platforms. However, each method allows passing in "platform-specifics" that contains data that is specific for customising notifications on each platform. This approach means that some scenarios may not be covered by the plugin. Developers can either fork or maintain their code for showing notifications in these situations. Note that the plugin still under development so expect the API surface to change over time.

Expand Down Expand Up @@ -453,7 +452,7 @@ When using custom notification sound, developers should be aware that iOS enforc

https://developer.apple.com/documentation/usernotifications/unnotificationsound?language=objc

**IMPORTANT**: There is an issue that prevents this plugin working properly with the `firebase_messaging` plugin at this point in time. This is being tracked [here](https://github.com/FirebaseExtended/flutterfire/issues/1455). Please upvote if this issue is important to you for the Flutter team to prioritise
**IMPORTANT**: There is an issue that prevents this plugin working properly with the `firebase_messaging` plugin at this point in time. This is being tracked [here](https://github.com/FirebaseExtended/flutterfire/issues/1455). Please upvote if this issue is important to you for the Flutter team to prioritise. I currently have a pull request to the `firebase_messaging` plugin waiting to be reviewed on this issue. For those that want to use my fork for their apps whilst waiting for the issue to be fixed, it can be found at https://github.com/MaikuB/flutterfire. See [here](https://github.com/MaikuB/flutter_firebase_local_notifications_examples/blob/55c3208c2bf19be487c5987be3661438a76529e1/java_objc/pubspec.yaml#L22) to see how it can be referenced in your application's `pubspec.yaml` file.

## Testing

Expand Down
2 changes: 1 addition & 1 deletion flutter_local_notifications/pubspec.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
name: flutter_local_notifications
description: A cross platform plugin for displaying and scheduling local notifications for Flutter applications with the ability to customise for each platform.
version: 1.1.7
version: 1.1.7+1
homepage: https://github.com/MaikuB/flutter_local_notifications/tree/master/flutter_local_notifications

dependencies:
Expand Down

0 comments on commit ac6563c

Please sign in to comment.