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

[TIMOB-19002] iOS9: Update Ti.Contacts to support new API #6945

Merged
merged 8 commits into from Jul 8, 2015

Conversation

cheekiatng
Copy link
Contributor

Ng Chee Kiat and others added 7 commits June 24, 2015 12:23
completed authorization request and check
completed showContacts
included new contacts delegates
First modifications to TiContactsPerson
added properties and labels in TiContactsPerson
added ‘reload’ listener
added getPersonByIdentifier and getGroupByIdentifier
updated getPeopleWithName
Added saveRequest property for deleting
implemented `select property` for `showContacts`
implemented getImage for TiContactPerson
createPerson
removePerson
createGroup
removeGroup
addMember
removeMember
getAllMembers and getSortedMembers
@hansemannn
Copy link
Collaborator

Results:

  • The app crashes, when you try to remove a contact which has already been removed.
  • SortedMembers (Ti.Contacts.CONTACTS_SORT_FIRST_NAME and Ti.Contacts.CONTACTS_SORT_LAST_NAME) are not working as mentioned before.

@cheekiatng Can you check that?

@cheekiatng
Copy link
Contributor Author

Yes the app crashes when trying to remove already removed contact. We have stated that in the docs here http://docs.appcelerator.com/platform/latest/#!/api/Titanium.Contacts-method-removePerson that this will result in unexpected behaviors. And unfortunately i couldn't figure a way to make it not crash either.

@hansemannn
Copy link
Collaborator

Crashs resolved, can be merged.

hansemannn added a commit that referenced this pull request Jul 8, 2015
[TIMOB-19002] iOS9: Update Ti.Contacts to support new API
@hansemannn hansemannn merged commit 8605329 into tidev:master Jul 8, 2015
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.

None yet

2 participants