Skip to content

Commit

Permalink
Adds support to request SiriKit permissions to platform interface. (#…
Browse files Browse the repository at this point in the history
…1266)

* Adding SiriKit permissions (#1140)

* Adding SiriKit permissions

* Fix tests

---------

Co-authored-by: Maurits van Beusekom <[email protected]>

* Reserves the assistant permission on no-op platforms

* Adds support to example application

* Remove pubspec overrides

* Fix formatting

* Revert to platforms to main

* Revert so only platform interface changes are included

* Update permission_handler_platform_interface/CHANGELOG.md

Co-authored-by: TimHoogstrate <[email protected]>

---------

Co-authored-by: Baptiste DUPUCH <[email protected]>
Co-authored-by: TimHoogstrate <[email protected]>
  • Loading branch information
3 people authored Jan 17, 2024
1 parent bb8e40a commit 0bf4d6f
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 2 deletions.
4 changes: 4 additions & 0 deletions permission_handler_platform_interface/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
## 4.1.0

* Adds the `Permission.assistant` which allows users to request permissions to access SiriKit on iOS and macOS platforms. This is a no-op on all other platforms.

## 4.0.2

* Improved the documentation around the `PermissionStatus`, `PermissionStatusGetters` and `FuturePermissionStatusGetters`.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -319,6 +319,10 @@ class Permission {
/// Permission for reading from and writing to the device's calendar.
static const calendarFullAccess = Permission._(37);

/// Android: Nothing
/// iOS: SiriKit
static const assistant = Permission._(38);

/// Returns a list of all possible [PermissionGroup] values.
static const List<Permission> values = <Permission>[
// ignore: deprecated_member_use_from_same_package
Expand Down Expand Up @@ -360,6 +364,7 @@ class Permission {
sensorsAlways,
calendarWriteOnly,
calendarFullAccess,
assistant,
];

static const List<String> _names = <String>[
Expand Down Expand Up @@ -401,6 +406,7 @@ class Permission {
'sensorsAlways',
'calendarWriteOnly',
'calendarFullAccess',
'assistant',
];

@override
Expand Down
2 changes: 1 addition & 1 deletion permission_handler_platform_interface/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ description: A common platform interface for the permission_handler plugin.
homepage: https://github.com/baseflow/flutter-permission-handler/tree/master/permission_handler_platform_interface
# NOTE: We strongly prefer non-breaking changes, even at the expense of a
# less-clean API. See https://flutter.dev/go/platform-interface-breaking-changes
version: 4.0.2
version: 4.1.0

dependencies:
flutter:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ void main() {
() {
const values = Permission.values;

expect(values.length, 38);
expect(values.length, 39);
});

test('check if byValue returns corresponding PermissionGroup value', () {
Expand Down

0 comments on commit 0bf4d6f

Please sign in to comment.