Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added SugarDataSource #580

Merged
merged 13 commits into from
Jun 24, 2016
Merged

Added SugarDataSource #580

merged 13 commits into from
Jun 24, 2016

Conversation

JonatanSalas
Copy link
Collaborator

Added SugarDataSource class which works on top of SugarRecord class and provides asynchronous execution of tasks using Futures and returns data via success and error callbacks.
This approach is good for those who use MVP pattern in their applications and this class can be combined with a Repository Pattern easily.

jonatan added 4 commits April 16, 2016 19:30
Added ThreadUtil class which provides a methoc to perform a generic async execution of a callable
Added SugarDataSource class which provides async execution of SugarRecord and result via callbacks.
Start adding SugarDataSourceTest to test all DataSource functionallity
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @nurolopher, @sibeliusseraphini and @whoshuu to be potential reviewers

@coveralls
Copy link

Coverage Status

Coverage increased (+2.1%) to 74.074% when pulling cd1949b on JonatanSalas:master into dec896a on satyan:master.

Added bulkInsert test and datasource constructor test
@coveralls
Copy link

Coverage Status

Coverage increased (+2.5%) to 74.492% when pulling c7d3961 on JonatanSalas:master into dec896a on satyan:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.5%) to 74.492% when pulling a716a34 on JonatanSalas:master into dec896a on satyan:master.

jonatan added 2 commits April 17, 2016 13:32
Modified from private to protected
@coveralls
Copy link

Coverage Status

Coverage increased (+2.6%) to 74.576% when pulling ae23fa6 on JonatanSalas:master into dec896a on satyan:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.6%) to 74.576% when pulling ae23fa6 on JonatanSalas:master into dec896a on satyan:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+3.7%) to 75.678% when pulling badf546 on JonatanSalas:master into dec896a on satyan:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+3.7%) to 75.678% when pulling 4059753 on JonatanSalas:master into dec896a on satyan:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+3.7%) to 75.678% when pulling d9bceb0 on JonatanSalas:master into dec896a on satyan:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+4.0%) to 75.915% when pulling 7f60eb9 on JonatanSalas:master into dec896a on satyan:master.

@sibelius
Copy link
Contributor

as this do not modify Sugar API, and LGTM, I'll merge it now

@sibelius sibelius merged commit 7434b98 into chennaione:master Jun 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants