Gaia/SMS: Difference between revisions

Jump to navigation Jump to search
8,184 bytes removed ,  9 June 2014
no edit summary
No edit summary
Line 9: Line 9:
https://mozilla.box.com/applications/1/864518430.
https://mozilla.box.com/applications/1/864518430.


== Use Cases ==
== Source Code ==
*Tom wants send a text message and selects the SMS app
The [https://github.com/mozilla-b2g/gaia/tree/master/apps/sms source code can be found in the gaia repository in github].
*Tom can send a new message by:
**Selecting an existing contact from the Contacts app list
**Entering a phone number
*Tom is notified of all incoming messages whether he's in the SMS app, on the Home Screen, or in a 3rd party app
*Tom also has the ability to send an MMS
**MMS supports photos and short videos
*Tom has the ability to search through his history of SMS sent/received
*Tom has the ability to delete specific SMS threads
*More user stories related to MMS in [https://bugzilla.mozilla.org/show_bug.cgi?id=842229 Bugzilla meta bug]


== Requirements ==  
== Filing bugs ==
;Gaia v1
You can file bug [https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox%20OS&component=Gaia%3A%3ASMS on Bugzilla]. Please include the Firefox OS version you use, and possibly a screenshot or a video.
*User has the ability to send/receive text messages
*User has the ability to send a new SMS:
**Enters a name based on available phone numbers in the Contacts app list
**Enters a phone number
*All text messages sent/received are threaded and displayed in a conversation-like view
*In the messages list view, timestamps are shown based on date of last received/sent messages
**'Today' and 'Yesterday' are used for recent message threads
*User has the ability to search through all messages sent/received
*User has the ability to delete specific message threads


;Gaia v2
== Project Management ==
*User has the ability to forward an individual or group of text messages to another recipient
We try to follow the Scrum methodology. Please have a look to the [[Gaia/SMS/Scrum|SMS Scrum Page]] for more information.
*User has the ability to send MMS messages
**MMS messages support photos and short videos


== Implementation Plan ( established April 18th, 2013) ==
== Some future development ==
* Milestone 1 - Completed by 4/26/2013
=== Using datastores for SMS ===
** SMIL, Multi-recipient, New layout and Gaia's pick activity
* [https://etherpad.mozilla.org/messages-datastore Etherpad]
* Milestone 2 - Completed by 5/10/2013
** Message composition with attachments, new text-display layout, open/save attachments
* Milestone 3 - Completed by 5/17/2013
** New thread layout, settings UI for automatic content download options
* Milestone 4/Final - Completed by 5/24/2013
** MMS Share activity (for other apps to send media to MMS), Notification of MMS


== Implementation Details: ==
=== Haida and Sheets ===
☆ - priority for merge to master,  ★ - in-progress, ✓ - landed on dev-branch or master, ⇪ - on master & uplifted
* [https://etherpad.mozilla.org/sms-haida Etherpad]
 
===Resolved===
 
* ✓⇪ New layout (Borja) <strike>{{bug|860680}}</strike> - LANDED, UPLIFTED
* ✓⇪ SMIL (Steve, Corey) {{bug|845173}}, LANDED, UPLIFTED
 
* ✓⇪ Activities for attaching media (Dominic)
** ✓⇪ Pick music {{bug|849766}} -  LANDED, UPLIFTED
** ✓⇪ Pick video <strike>{{bug|849768}}</strike> - LANDED, UPLIFTED
 
* Device Timestamp -
** <strike>{{bug|840051}}</strike> - RESOLVED, NO CHANGES
* Media Preview (Steve/Corey)
** ✓⇪ <strike>{{bug|840056}}</strike> - audio playback, LANDED, UPLIFTED
** ✓⇪ <strike>{{bug|840057}}</strike> - video playback, LANDED, UPLIFTED
** ✓⇪ <strike>{{bug|840059}}</strike> - photo display, LANDED, UPLIFTED
 
** ✓ New Thread Layout
*** ✓ {{bug|840055}} (Mike) - LANDED, not uplifted
 
* Thread List
** ★ <strike>{{bug|862311}}</strike> (Mike) - paperclip icon (has WIP patch, needs asset)
*** depends on:
**** ✓ <strike>gecko bug {{bug|863241}}</strike> is landed in mozilla-central, and
**** ✓ <strike>{{bug|865287}}</strike> which is landed and awaiting uplift
 
* Media Saving (DJF)
** {{bug|840065}} meta-bug
** ✓⇪ <strike>{{bug|867231}}</strike> support saving previewed media in Gallery app
** ✓⇪ <strike>{{bug|867448}}</strike> support saving previewed media in Music app
** ✓⇪ <strike>{{bug|869292}}</strike> support saving previewed media in Video app
*** ✓⇪ <strike>{{bug|868625}}</strike> - CRASH, blocking these save bugs from moving forward. Bent fixed.
 
* Settings UI
** ✓⇪ <strike>{{bug|840076}}</strike> (Steve) - MMS download options
** ✓⇪ <strike>{{bug|842487}}</strike> (Steve) - Download option settings page layout
** ✓⇪ <strike>{{bug|842251}}</strike> (Steve) - SMS delivery report, Gaia parts
*** ✓ <strike>{{bug|863130}}</strike> (Jaoo) - SMS delivery report, Gecko parts, LANDED NOT UPLIFTED
 
* Message Expiry
** ✓⇪ <strike>{{bug|867227}}</strike> (Chia-hung) - Add expiry date into nsIDOMMozMmsMessage
 
* Notification of MMS (Steve)
** <strike>{{bug|840049}}</strike> New message notification
** <strike>{{bug|840050}}</strike> Messages app invoke from notification
 
===Milestone 2 (5/10/13)===
 
* Composer
** Composition to handle media (Greg):
*** ✓ <strike>{{bug|840069}}</strike> message preview, landed on dev branch
** Attachment Support (Greg)
*** ☆★ {{bug|870057}} (Greg) - maybe Gecko problem
** Carrier Limit
*** ✓⇪ <strike>{{bug|840061}}</strike> (Steve) - operator limit definition
*** ☆ {{bug|870120}} max segments info no longer working
*** ☆ <strike>{{bug|870124}}</strike> message counter styling
*** ☆ <strike>{{bug|870536}}</strike> 'enableSend' method is not working as expected
** ☆ {{bug|870628}} Text in input box can overflow below the interface. Regression.
 
* ★ Multi-recipient (Rick) <strike>{{bug|837994}}</strike>
** ☆ {{bug|870601}} z-index problem on recipient search panel
** ☆ {{bug|870603}} (Rick) Contact entry bugs with ; and ENTER
** ☆ <strike>{{bug|870618}}</strike> (Steve) Live-search panel is not cleaned after sending a SMS
** ✓ <strike>{{bug|870879}}</strike> New composer should convert &lt;br> to \n instead of stripping spaces
 
===Milestone 3 (5/17/13)===
 
* Open (pushed to Milestone 4, below)
** {{bug|868218}} (Corey) Handle not-downloaded messages in thread view. Patches in review.
** {{bug|868227}} (Greg) Attachment (pre-send). Patches in review.
 
* Closed
** <strike>{{bug|868679}}</strike> (Rick) use threadId as the "key" of a thread instead of a phone number. Patch waiting on review from Corey, Julien.
*** Blocked by <strike>{{bug|870562}}</strike> (fixed on m-c, uplifted)
*** Also fixes <strike>{{bug|870145}}</strike> Sending to N>1 recipients should create individual threads and return to thread-list.
** <strike>{{bug|870416}}</strike> (Rick) Recipients container interaction. Drag & Pull effects. (dupe of {{bug|870544}})
** <strike>{{bug|840087}}</strike> (Steve) New message notification when auto-download disabled (Only need copywriter to provide l10n string. The priority could be lower since we already has temp string here)
** <strike>{{bug|840075}}</strike> (Mike) MMS "share" activity for images (Landed in master on Thurs.)
** <strike>{{bug|840044}}</strike> (Mike) Image Attachment Support (Landed in master on Tues.)
** <strike>{{bug|869255}}</strike> (Mike)  Attach button fire pick Activity and create Attachment (PR for UI already up for review. supporting logic in for review Monday evening.) (Landed in master on Tues.)
** <strike>{{bug|840090}}</strike> (Steve) New message notification when auto-download enabled. (Landed in master on Tues.)
** <strike>{{bug|840035}}</strike> (Corey) UI: operator defined limit prompt (Landed in master on Weds.)
** <strike>{{bug|867440}}</strike> (Chia-hung) Add more delivery status for delivery state "not-downloaded" and send the dom message with right delivery status.
** <strike>{{bug|869219}}</strike> (Mike) Reset handler is bound unsafely (Landed in master on Weds.)
** <strike>{{bug|870544}}</strike> (Mark) Apply right UI styles (Landed in master on Fri)
** <strike>{{bug|870164}}</strike> (Steve) Multi-Recip container not taking all available height (Set dup to 870544 first, reopen if necessary)
** <strike>{{bug|869717}}</strike> (Steve) CSS tweaks needed for MMS placeholders.(Landed in master on Sat)
 
===Milestone 4 (5/24/13)===
 
'''Pushed from M3, and new blockers
* <strike>{{bug|868218}}</strike> (Corey) Handle not-downloaded messages in thread view.
* <strike>{{bug|868227}}</strike> (Greg) Attachment (pre-send).
* {{bug|874186}} (Vicamo) After sending a new message to a new contact, ensure headers update properly
* <strike>{{bug|872369}}</strike> (Corey) Send MMS messages via new sendMMS
* {{bug|873145}} (Ctai) sendMMS does not transmit end-to-end. has patch, has review, needs landing
* <strike>{{bug|870057}}</strike> (Steve) Input field in bottom bar is not working as expected. has patch, has review, needs landing
* <strike>{{bug|873477}}</strike> (Rick) Group participants string formatting update per v8.0
 
''' [https://bugzilla.mozilla.org/buglist.cgi?quicksearch=sw%3Auplift-blocker&list_id=6622567 Remaining uplift blockers]
 
'''Nice to have
* {{bug|810099}} (Chia-hung) support onretrieving event
* {{bug|874912}} () SmsMessage interface should expose a receivers property
 
''' Existing M4 plan
* Visual design meta-bug {{bug|872514}}
 
== Meeting minutes ==
* [https://etherpad.mozilla.org/SeUu340ZGQ MMS meeting (March 15, 2013)]
* [https://etherpad.mozilla.org/MMS-v1-1 Taipei office weekly MMS meeting]
 
== Development ==
[https://github.com/mozilla-b2g/gaia/tree/master/apps/sms Source code on Github]
 
In progress code will be landed to the [http://github.com/bocoup/gaia/tree/messaging messaging branch on Bocoup's fork on Github] for the sake of keeping friction on gaia/master minimal.


== Security Review ==
== Security Review ==
The security review for this app can be found [https://wiki.mozilla.org/Security/Reviews/Gaia/sms here].
The security review for this app can be found [https://wiki.mozilla.org/Security/Reviews/Gaia/sms here].
Confirmed users
821

edits

Navigation menu