{"action":"created","issue":{"url":"https://api.github.com/repos/satyan/sugar/issues/556","repository_url":"https://api.github.com/repos/satyan/sugar","labels_url":"https://api.github.com/repos/satyan/sugar/issues/556/labels{/name}","comments_url":"https://api.github.com/repos/satyan/sugar/issues/556/comments","events_url":"https://api.github.com/repos/satyan/sugar/issues/556/events","html_url":"https://github.com/satyan/sugar/pull/556","id":144721473,"number":556,"title":"Major modifications: Addition of ContentProvider and column name changed from id to _id.","user":{"login":"bpappin","id":3903833,"avatar_url":"https://avatars.githubusercontent.com/u/3903833?v=3","gravatar_id":"","url":"https://api.github.com/users/bpappin","html_url":"https://github.com/bpappin","followers_url":"https://api.github.com/users/bpappin/followers","following_url":"https://api.github.com/users/bpappin/following{/other_user}","gists_url":"https://api.github.com/users/bpappin/gists{/gist_id}","starred_url":"https://api.github.com/users/bpappin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bpappin/subscriptions","organizations_url":"https://api.github.com/users/bpappin/orgs","repos_url":"https://api.github.com/users/bpappin/repos","events_url":"https://api.github.com/users/bpappin/events{/privacy}","received_events_url":"https://api.github.com/users/bpappin/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/satyan/sugar/labels/PR:%20Needs%20Rebase","name":"PR: Needs Rebase","color":"d93f0b"},{"url":"https://api.github.com/repos/satyan/sugar/labels/PR:%20Needs%20Review","name":"PR: Needs Review","color":"fbca04"},{"url":"https://api.github.com/repos/satyan/sugar/labels/tests%20Needed","name":"tests Needed","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":30,"created_at":"2016-03-30T21:13:28Z","updated_at":"2016-05-29T07:38:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/satyan/sugar/pulls/556","html_url":"https://github.com/satyan/sugar/pull/556","diff_url":"https://github.com/satyan/sugar/pull/556.diff","patch_url":"https://github.com/satyan/sugar/pull/556.patch"},"body":"These are some major modifications in order to support several new features, most important among them, a new **ContentProvider**, so you can work with **CursorLoader** and the **CursorAdapters**.\r\n\r\nThis code is *not* compatible with version 1.5 of Sugar, mainly because the `id` column has been changed to `_id` as defined in `android.provider.BaseColumns._ID` _(should have been like that from the start IMO)_ and I don't expect it to be merged. \r\n\r\nAt this point I am just making sure I share the code back to the project."},"comment":{"url":"https://api.github.com/repos/satyan/sugar/issues/comments/222347318","html_url":"https://github.com/satyan/sugar/pull/556#issuecomment-222347318","issue_url":"https://api.github.com/repos/satyan/sugar/issues/556","id":222347318,"user":{"login":"shobhik","id":1535459,"avatar_url":"https://avatars.githubusercontent.com/u/1535459?v=3","gravatar_id":"","url":"https://api.github.com/users/shobhik","html_url":"https://github.com/shobhik","followers_url":"https://api.github.com/users/shobhik/followers","following_url":"https://api.github.com/users/shobhik/following{/other_user}","gists_url":"https://api.github.com/users/shobhik/gists{/gist_id}","starred_url":"https://api.github.com/users/shobhik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shobhik/subscriptions","organizations_url":"https://api.github.com/users/shobhik/orgs","repos_url":"https://api.github.com/users/shobhik/repos","events_url":"https://api.github.com/users/shobhik/events{/privacy}","received_events_url":"https://api.github.com/users/shobhik/received_events","type":"User","site_admin":false},"created_at":"2016-05-29T07:38:41Z","updated_at":"2016-05-29T07:38:41Z","body":"Hi, not to sound ungrateful/whiny but when can we expect this to be merged successfully, or see a 2.0 release? I ask because I haven't seen any activity on this thread for a while so I have no idea what timeframe is involved.\r\n\r\nWe have ~4 apps using Sugar that will enter production in the near future. Data loss being a big deal in our industry, it would make all our lives easier if we didn't have to force our clients to reinstall between 1.x and 2.0 so I'd rather wait to release our apps with Sugar 2.0. \r\n\r\n@bpappin @JonatanSalas I'm not the greatest at unit testing but am willing to do physical QA (time permitting) if that will expedite things.\r\n\r\nGoes without saying I love this library and the hard work y'all have put into it!"}}
{"actor":{"display_login":"shobhik"}}