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

[TIDOC-399] APIDoc: UI.EmailDialog rewrite #1470

Merged
merged 7 commits into from Feb 25, 2012
Merged

Conversation

hal-gh
Copy link
Contributor

@hal-gh hal-gh commented Feb 21, 2012

@hal-gh
Copy link
Contributor Author

hal-gh commented Feb 21, 2012

Please review

@sptramer
Copy link
Contributor

Will CR

add an attachment to the email. The attachment can either be a Blob or File object. Note: Versions before 1.7.1 will only support one attachment in Android. Since version 1.7.1, our Android implementation will accept multiple attachments, but multiple attachments will only work if the user chooses to send via the Android Gmail app; the Android "Mail" app (which is anyway not available on some handsets) will only accept one attachment from a Titanium app. K9 has also been tested successfully with multiple attachments.
summary: Adds an attachment.
description: |
On Android, since version 1.7.1, multiple attachments may be added, but will only work if
Copy link
Contributor

Choose a reason for hiding this comment

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

Considering that users reading this document will almost certainly not be using 1.7.1 do we need this in here? This is a revision history outside of since, not a description of the API.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@sptramer
Copy link
Contributor

Docs reviewed, please address comments

@hal-gh
Copy link
Contributor Author

hal-gh commented Feb 24, 2012

Updated. Please review.

@sptramer
Copy link
Contributor

Docs reviewed. APPROVED.

summary: object of animation properties. pass `animated` property (as boolean) to indicate if the dialog should be animated on open.
summary: |
Animation properties. Set `animated` Boolean property to `true` to animate dialog
on open.
type: Object
Copy link
Contributor

Choose a reason for hiding this comment

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

open does not take any parameters in android

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in next PR.

@ayeung
Copy link
Contributor

ayeung commented Feb 24, 2012

docs reviewed. Had a couple of comments

@hal-gh
Copy link
Contributor Author

hal-gh commented Feb 25, 2012

Updated. Please review.

@ayeung
Copy link
Contributor

ayeung commented Feb 25, 2012

Code reviewed, left one more comment

@hal-gh
Copy link
Contributor Author

hal-gh commented Feb 25, 2012

Updated. Please review.

- name: SENT
summary: constant for the SENT status result
summary: Constant for the `SENT` status result.
description: On Android, this property exists but is not used.
type: Number
permission: read-only
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 actually reversed. SENT and FAILED used in android. CANCELLED and SAVED are not used. We will also want to remove the [iphone, ipad] tag from CANCELLED and SAVED properties

@hal-gh
Copy link
Contributor Author

hal-gh commented Feb 25, 2012

Updated. Please review.

@ayeung
Copy link
Contributor

ayeung commented Feb 25, 2012

Code reviewed. Request Accepted

@ayeung
Copy link
Contributor

ayeung commented Feb 25, 2012

Functionally reviewed. Request Accepted

marshall added a commit that referenced this pull request Feb 25, 2012
[TIDOC-399] APIDoc: UI.EmailDialog rewrite
@marshall marshall merged commit 3fdade2 into tidev:master Feb 25, 2012
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

4 participants