Skip to content
This repository has been archived by the owner on Mar 14, 2023. It is now read-only.

Added functionality to simplify the initialisation of the wrapper. #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pkclsoft
Copy link

These changes were primarily to facilitate Shared iPad support.

I've also set the synchronize to happen more often because the delay was too great (IMO).

These changes were primarily to facilitate Shared iPad support.
@@ -134,6 +134,20 @@
*/
+(void)removeNotifications;

#define ICLOUD_DATA_ENABLED_KEY @"iCloudDataEnabled"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer constants to be defined using the extern NSString* const syntax, so the actual value can be hidden in the .m file. (See here for an example.)

@sdarlington
Copy link
Owner

Incidentally, I deliberately left calling synchronise to the host app as it's not quite as simple as "more often is better." When using the iCloud version, calls get throttled if you use it too often, which results in the save happening less often.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants