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-26032] iOS/Android: Fix Ti.Contacts parity, deprecate old API's properly #10038
Conversation
Generated by 🚫 dangerJS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The deprecation note on Android has a type in the correct method name. Added suggested fixed text.
@@ -136,6 +136,13 @@ public void save(Object people) | |||
|
|||
@Kroll.method | |||
public PersonProxy getPersonByID(long id) | |||
{ | |||
Log.w(TAG, "getPersonByID has been deprecated in favor of getPersonById (parity)", Log.DEBUG_MODE); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Log.w(TAG, "Ti.Contacts.getPersonByID() has been deprecated in favor of Ti.Contacts.getPersonByIdentifier() for cross-platform parity", Log.DEBUG_MODE);
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated!
Generated by 🚫 dangerJS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR passed.
Note - @hansemannn There seems linting issue in Contacts.java as build is failing. Can you fix that.
FR Passed. |
JIRA: https://jira.appcelerator.org/browse/TIMOB-26032
no tests
because it only refactors existing code. Also requires UI-related interaction (contacts permissions, selecting values from the modal dialog).