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

*Major* rebase #5

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

*Major* rebase #5

wants to merge 17 commits into from

Conversation

a2
Copy link

@a2 a2 commented Jan 5, 2012

No description provided.

Signed-off-by: Alexsander Akers <[email protected]>
Signed-off-by: Alexsander Akers <[email protected]>
@iosdeveloper iosdeveloper mentioned this pull request Jan 7, 2012
@iosdeveloper
Copy link

My thoughts and opinion on this class after testing the whole day:

  • What about keys that are already in the Cloud when an app is installed on another device or reinstalled?
  • Wouldn't it be great to have the opinion to exclude some keys especially the WebKit-ones or device-specific ones.
  • What would you do when Cloud is full of data and you want to clean it (for testing)?

I'm having solutions for all tree I'm now using in my version of iCloudSync class. What do you think about these?

@a2
Copy link
Author

a2 commented Jan 8, 2012

@iosdeveloper That looks good. I can't seem to find your branch anymore, though… :/

@iosdeveloper
Copy link

Its set to private for the moment.

@a2
Copy link
Author

a2 commented Jan 8, 2012

@iosdeveloper Okay. Perhaps you could share it with me so I could see? You can always set it to private but not push your changes until they're all done.

@iosdeveloper
Copy link

Alright. It is public. Please let me know what youre thinking. http://github.com/iosdeveloper/MKiCloudSync/commit/db14cb2b7716c9ca1c912872e37f0f86307e410f

@a2
Copy link
Author

a2 commented Jan 8, 2012

I like it a lot. It does need some work though, like an API for setting ignored keys. Send me a pull request and I'll merge it in! I'll edit the formatting to match my "personal guidelines" though. :D

iosdeveloper and others added 10 commits January 8, 2012 01:49
@a2
Copy link
Author

a2 commented Jun 4, 2012

Is this pull request ever going to happen? :/

a2 added 3 commits July 8, 2012 07:02
Change so that +start can be called multiple times
Signed-off-by: Alexsander Akers <[email protected]>
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.

3 participants