search

itsniper / FTASync

Allows you to sync CoreData entities with a Parse backend.
MIT License
311 stars 59 forks source link

Added support for classes deeper than immediate subclasses of FTASyncParent #2

Closed AndrewSmall closed 12 years ago

itsniper commented 12 years ago

Awesome thanks! I'll try and take a look this weekend.

AndrewSmall commented 12 years ago

Thanks to you!

The next thing would be to add support for Parse PFRelations. I saw a TODO in there about upgrading to a new Parse SDK for that.

Andrew

On Thu, Sep 20, 2012 at 12:32 PM, Justin Bergen notifications@github.comwrote:

Awesome thanks! I'll try and take a look this weekend.

— Reply to this email directly or view it on GitHubhttps://github.com/itsniper/FTASync/pull/2#issuecomment-8740851.

itsniper commented 12 years ago

Yeah, the included SDK has it in there. But it was added after I was already setup with doing relationships the "old" way.

itsniper commented 12 years ago

Merged in the pull request on Saturday, but now sync is not working properly. So I'm working on debugging that. Hopefully can get it resolved soon.

itsniper commented 12 years ago

One issue ended up being an object ID issue in Magical Record 2. Adding a todo item to a person created during this run session was causing a "ghost" person managedObject. Pulling in a more recent version of MR (magicalpanda/MagicalRecord@abb7314b26877982f44933998782056f43d61b8c) seems to have fixed it. I believe it was the permanent id fixes put in MR on 9/28.

AndrewSmall commented 12 years ago

Thanks Justin. Do you have any idea how big is a migration to the new PFRelations from the old style? I am not familiar with the old style.

Andrew

On Mon, Oct 1, 2012 at 6:18 PM, Justin Bergen notifications@github.comwrote:

One issue ended up being an object ID issue in Magical Record 2. Adding a todo item to a person created during this run session was causing a "ghost" person managedObject. Pulling in a more recent version of MR (commit abb7314https://github.com/itsniper/FTASync/commit/abb7314b26877982f44933998782056f43d61b8c) seems to have fixed it. I believe it was the permanent id fixes put in MR on 9/28.

— Reply to this email directly or view it on GitHubhttps://github.com/itsniper/FTASync/pull/2#issuecomment-9054918.

itsniper commented 12 years ago

It may be a fair bit of work to do it properly. The biggest issue is going to be the conflict resolution code. As I was reminded today, that code is fairly complex and fragile. I opened Issue #3 for this, so we can move the convo over there.

itsniper commented 12 years ago

Fixed the final issue. Adding support for custom data class names had broken the relationship conflict resolution. Fixed with commit 2423154084a496c08f8912cb8e9f7381e9fbe012 .