Features/Thunderbird/BigFiles: Difference between revisions

no edit summary
No edit summary
No edit summary
 
(7 intermediate revisions by 2 users not shown)
Line 1: Line 1:
{{FeatureStatus
{{FeatureStatus
|Feature name=Big Files
|Feature name=Big Files
|Feature stage=Planning
|Feature stage=Shipped
|Feature status=In progress
|Feature status=In progress
|Feature version=Thunderbird 13
|Feature version=Thunderbird 13
Line 62: Line 62:


We are also not, for this iteration, supporting the creation of multiple accounts with the same service provider. This will be addressed in a later version.
We are also not, for this iteration, supporting the creation of multiple accounts with the same service provider. This will be addressed in a later version.
|Feature security review=[https://wiki.mozilla.org/Security/Reviews/Thunderbird/BigFiles]
|Feature privacy review=[https://wiki.mozilla.org/Privacy/Reviews/ThunderbirdBigFiles]
|Feature implementation notes=The bug # for this feature is [https://bugzilla.mozilla.org/show_bug.cgi?id=698925 bug 698925]
|Feature implementation notes=The bug # for this feature is [https://bugzilla.mozilla.org/show_bug.cgi?id=698925 bug 698925]


Line 146: Line 148:
* <strike>Link underlines in HTML email don't match link colour (mconley)</strike>(Done March 6th)
* <strike>Link underlines in HTML email don't match link colour (mconley)</strike>(Done March 6th)
* <strike>Fix YouSendIt recipients (currently pointing to dbienvenu) (mconley)</strike> (Done March 6th)
* <strike>Fix YouSendIt recipients (currently pointing to dbienvenu) (mconley)</strike> (Done March 6th)
* Saving as Drafts bugs
* <strike>Need to put Dropbox API key into production mode to increase testing exposure (bienvenu)</strike>
** Attachment not listed in bucket on attach and save on draft
* <strike>Need to handle saving as draft and reloading drafts when there are cloud attachments. (bienvenu)</strike>
*** Create a new message, and attach a cloud file
* <strike>Saving as Drafts bugs</strike>
*** Save as a draft and close the compose window
** <strike>Attachment not listed in bucket on attach and save on draft (bienvenu)</strike>
*** Reopen the message, and attach another cloud file.
*** <strike>Create a new message, and attach a cloud file</strike>
*** Save as a draft and close the compose window
*** <strike>Save as a draft and close the compose window</strike>
*** Reopen the message - the second file is not listed in the bucket. Icon in inserted HTML URL might be broken.
*** <strike>Reopen the message, and attach another cloud file.</strike>
*** <strike>Save as a draft and close the compose window</strike>
*** <strike>Reopen the message - the second file is not listed in the bucket. Icon in inserted HTML URL might be broken.</strike>
* <strike>Improve activity feedback after manually authorizing in preferences dialog (mconley)</strike>
* <strike>Investigate weird behaviour with mixed attachment types</strike>
** <strike>Observed weird behaviour is the normal attachments being renamed on the receiver end</strike>
* <strike>Switch from global attachment nsIObserver events to window-specific DOM events (squib)</strike>
 
''Post-Landing:''
* Security and Privacy Reviews (Feb 29th-ish)  
* Security and Privacy Reviews (Feb 29th-ish)  
* Part of the notification text could be a link to an external "What's this" page.
* Spinner doesn't give enough information.  (Mar 9th)
** We can't determine upload progress using nsIRequestObserver, so satisfying this requirement would likely involve changing the nsIMsgCloudFileProvider interface, along with a bunch of our code. [https://bugzilla.mozilla.org/show_bug.cgi?id=736169 bug 736169]
* Part of the notification text could be a link to an external "What's this" page. [https://bugzilla.mozilla.org/show_bug.cgi?id=737348 bug 737348]
* Test that URL insertions happen in expected locations [https://bugzilla.mozilla.org/show_bug.cgi?id=736055 bug 736055]
* Fix layout for management panes on OSX [https://bugzilla.mozilla.org/show_bug.cgi?id=736155 bug 736155]
* Convert between different cloud providers  (mostly done)
* Convert between different cloud providers  (mostly done)
** Need to remove the old html link before adding the new one.
** Need to remove the old html link before adding the new one.
Line 161: Line 175:
* Whatever comes up from Anne-Marie's Milestone 2 User Testing.  (Unknown until we get more data.)
* Whatever comes up from Anne-Marie's Milestone 2 User Testing.  (Unknown until we get more data.)
* I10n sweep (in particular, the Dropbox and YouSendIt management / settings iframe documents)
* I10n sweep (in particular, the Dropbox and YouSendIt management / settings iframe documents)
* TODO / XXX / FIXME sweep
** components/cloudfile/nsDropbox.js:
***// XXX: replace this with a better function
** components/cloudfile/nsYouSendIt.js
***// XXX: replace this with a better function
** components/cloudfile/cloudFileAccounts.js
***// TODO: sort by ordinal
***// Pick a unique key (TODO: this is a dumb way to do it, probably)
** components/preferences/applications.js
***// TODO: What other states are there?
***// TODO Handle this case
** components/preferences/applications.xul
***<strike>TODO - The panes below need more thought (mconley)</strike> (Done March 6th)
** locales/en-US/chrome/messenger/preferences/applications.dtd
***TODO: Access keys
** components/compose/content/cloudAttachmentLinkManager.js
***<strike>// TODO we need a place to send the user (mconley)</strike> (Done March 6th)
***// XXX find a better spot to insert this
** components/compose/content/MsgComposeCommands.js
***// XXX use a generic "cloud" icon here
***// XXX support actions other than Upgrade
***// XXX: handle drafts (originalUrl won't be defined)
** themes/gnomestripe/mail/compose/messengercompose.css
***XXX: remove this probably?
* Revisit themeing / colours for inserted attachments in HTML mail
* Revisit themeing / colours for inserted attachments in HTML mail
* Improve activity feedback after manually authorizing in preferences dialog
* Investigate using something other than the modal alert box on uploading/delete errors in the compose window
* Investigate using something other than the modal alert box on uploading/delete errors in the compose window
* Fix layout for management panes on OSX
* Need to put Dropbox API key into production mode to increase testing exposure
* Investigate weird behaviour with mixed attachment types
** Observed weird behaviour is the normal attachments being renamed on the receiver end
* Canceling a message with cloud file attachments should delete the uploaded files from the server (may need to special case drafts and templates)
* Canceling a message with cloud file attachments should delete the uploaded files from the server (may need to special case drafts and templates)
* Need to handle saving as draft and reloading drafts when there are cloud attachments. (bienvenu)
* Switch from global attachment nsIObserver events to window-specific DOM events (squib)
* Test that URL insertions happen in expected locations
* Use a better graphic to indicate "loading" in the main selected item panel in the preferences dialog when loading account information.
* Use a better graphic to indicate "loading" in the main selected item panel in the preferences dialog when loading account information.
* Spinner doesn't give enough information.  (Mar 9th)
** We can't determine upload progress using nsIRequestObserver, so satisfying this requirement would likely involve changing the nsIMsgCloudFileProvider interface, along with a bunch of our code.
* Some tests are missing.  (As many as we can write before the deadline.)
* Some tests are missing.  (As many as we can write before the deadline.)
** A lot of the UI tests.
** A lot of the UI tests.
** No fake provider.
** No fake provider.
** Testing with accounts other than mconley's.
** <strike>Testing with accounts other than mconley's.</strike>
** External service starts sending garbage.
** External service starts sending garbage.
** We can't log in for some reason
** We can't log in for some reason
Line 208: Line 188:
|Feature security health=OK
|Feature security health=OK
|Feature security notes=[[Security/Reviews/Thunderbird/BigFiles|Notes]]
|Feature security notes=[[Security/Reviews/Thunderbird/BigFiles|Notes]]
|Feature privacy notes=[[Privacy/Reviews/ThunderbirdBigFiles|Privacy/Reviews/ThunderbirdBigFiles]]
}}
}}
<div style="margin: 5px 0px 5px 0px; width: 300px; border: 1px solid green; padding: 5px; background-color: #CFC; text-align: center;">[[Talk:Features/Thunderbird/BigFiles | Contribute to this feature. ]]</div>
<div style="margin: 5px 0px 5px 0px; width: 300px; border: 1px solid green; padding: 5px; background-color: #CFC; text-align: center;">[[Talk:Features/Thunderbird/BigFiles | Contribute to this feature. ]]</div>
canmove, Confirmed users, Bureaucrats and Sysops emeriti
3,628

edits