Add CloudKit discoverability permissions.

[evermeer]

For adding CloudKit discoverability permissions you could use this code. You do have to be aware that this is an async call.

        let container = CKContainer.default()
        container.status(forApplicationPermission: CKApplicationPermissions.userDiscoverability, completionHandler: { applicationPermissionStatus, error in
                 // handle applicationPermissionStatus for statuses like
                 // CKApplicationPermissionStatus.Granted, .Denied, .CouldNotComplete, .InitialState
        })


        let container = CKContainer.default()
        container.accountStatus(completionHandler: {status, error in
            switch status {
            case .available, .restricted:
                container.requestApplicationPermission(CKApplicationPermissions.userDiscoverability,  completionHandler: { applicationPermissionStatus, error in
                          // handle applicationPermissionStatus for statuses like CKApplicationPermissionStatus.Granted, .Denied, .CouldNotComplete, .InitialState
                })
            case .couldNotDetermine, .noAccount: break
                // Ask user to login to iCloud
            }
        })

[ennioma]

Thanks for the suggestion, I'll add it soon

[ennioma]

Hi @evermeer,
I've added ArekCloudKit in the branch https://github.com/ennioma/arek/tree/addCloudKitPermission. It's not actually merged to develop because I need to test it more.

I'd love to hear your feedback on this branch, I've just integrated your example into Arek.

Thanks!

[evermeer]

Thanks @ennioma I will give it a test drive this weekend

[ennioma]

@evermeer did you have any chance to test the update on that branch? Thanks!

[evermeer]

Yes, I did change some code and made a pull request for it. For managing the permitions you need 2 calls. The 1st is to see if the user is loged in into iCloud and the second is to ask for discoverability permissions. See the PR

[ennioma]

Sorry I missed that :)

I take a look soon!

[ennioma]

PR #48 merged.