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-14218: Android: Implement apiName constants #4826

Merged
merged 4 commits into from Oct 23, 2013

Conversation

pingwang2011
Copy link
Contributor

@ghost ghost assigned hieupham007 Oct 22, 2013
summary: The name of the API that this proxy corresponds to.
description: |
The value of this property is the fully qualified name of the API. For example, a [Button](Titanium.UI.Button)
will return "Ti.UI.Button" for apiName.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change last sentence to:

For example, Button returns Ti.UI.Button.

@bhatfield
Copy link
Contributor

Reviewed. Ran validate and docgen. Left one comment.

returns `Ti.UI.Button`.
permission: read-only
type: String
platforms: [android, blackberry, iphone, ipad]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is already implemented on blackberry and ios?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Checked the subtasks https://jira.appcelerator.org/browse/TIMOB-12553. Actually it's also done on MW and Tizen. Will update the doc.

@hieupham007
Copy link
Contributor

Code reviewed and functionally tested. Request accepted

hieupham007 added a commit that referenced this pull request Oct 23, 2013
timob-14218: Android: Implement apiName constants
@hieupham007 hieupham007 merged commit e91102a into tidev:master Oct 23, 2013
@pingwang2011 pingwang2011 deleted the timob-14218 branch December 3, 2013 18:24
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

3 participants