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

[Contact service android] Split the projection for contact info. Allow apps to configure the contact info they wants. #187

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

Conversation

zhenqiu1101
Copy link

Querying all contact columns results in the error: "Row too big to fit into CursorWindow". That is because some columns are referring to the same internal column (data1, data2 .... data6 etc). We get duplicate columns for single row. So we split the cursors for querying different contact info. Add an optional parameter for apps to configure what contact info they wants.

@zhenqiu1101
Copy link
Author

@lukasgit Hi Lukas, can you take a look at this PR?

@lukasgit
Copy link
Owner

@zhenqiu1101 will do, thanks

@lukasgit
Copy link
Owner

@lukasgit Hi Lukas, can you take a look at this PR?

Apologies for the delay @zhenqiu1101 . Is this PR tested are ready? Is there anything instructions needed to be added to the readme ?

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

Successfully merging this pull request may close these issues.

2 participants