840515 - (mms-by-email) [User Story][Messages] Send to Email address (FxOS 2.2)

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect, P1)

Description

[Peter Dolanjski [:pdol]]

UCID: Messages-030

User Story:
As a user, I can compose an MMS message to an email address

Comment 1

[Dylan Oliver [:doliver]]

Clearing tracking-b2g18 flag from User Story bugs. This flag is for bugs that we would take fixes for in the 1.x branch. Feature work should be officially slotted into a release instead with leo+. If this story is intended for v1.1, please nominate for leo? blocking.

Comment 2

[Wayne Chang [:wchang]]

As is now, this will not be within the v1.1 MMS scope.

Comment 5

[Wilfred Mathanaraj [:WDM]]

Moving the acceptance Criteria over to this bug:

Acceptance Criteria:
* Send a MMS (image, audio, text and video) to an e-mail account, verify that it correctly arrives to that e-mail account as an attachment
* Send a MMS to multiple recipients:
** only e-mail accounts
** e-mail accounts and phone numbers 
*** verify that it correctly arrives to all the recipients

Comment 6

[Joe Cheng [:jcheng] (please needinfo)]

add to backlog 891754

Comment 8

[xiupinglong]

Dears,

Since this will not be within the v1.1 MMS scope, when do you think will have this?

Comment 9

[Wilfred Mathanaraj [:WDM]]

this is one of the target items at the moment for v1.3

Comment 10

[Beatriz Rodríguez [:brg]]

(In reply to Wilfred Mathanaraj [:WDM] from comment #9)
> this is one of the target items at the moment for v1.3
I would like to vote for this feature to be added soon in the roadmap. Please avoid any delay after v1.3.
We had received this request from some carriers. Thanks.

Comment 11

[Rick Waldron [:rwaldron]]

Is support already implemented in mozMobileMessage.sendMMS? Once that's complete, this should be trivial.

Comment 12

[Gene Lian (I already quit Mozilla)]

(In reply to Rick Waldron from comment #11)
> Is support already implemented in mozMobileMessage.sendMMS? Once that's
> complete, this should be trivial.

Bug 866938 is under way to support. Gaia will be able to directly call sendMMS with e-mail addresses.

Comment 13

[Rick Waldron [:rwaldron]]

(In reply to Gene Lian [:gene] from comment #12)
> (In reply to Rick Waldron from comment #11)
> > Is support already implemented in mozMobileMessage.sendMMS? Once that's
> > complete, this should be trivial.
> 
> Bug 866938 is under way to support. Gaia will be able to directly call
> sendMMS with e-mail addresses.

This is great news, thank you.

Comment 14

[Paul Yang [:pyang] (away)]

Change in-moztrap and start to work on test cases

Comment 15

[Paul Yang [:pyang] (away)]

Considering to use email from contact, need ux input for this case.
need info? Ayman

Comment 16

[ayman maat :maat]

(In reply to Paul Yang: pyang: [e-mail] from comment #15)
> Considering to use email from contact, need ux input for this case.
> need info? Ayman

Hey Paul, happy to give you input, can you give me more detail on the input you would like.

Thanks

Comment 17

[Paul Yang [:pyang] (away)]

Hi Ayman,
just want to make sure if we cover these cases:
- email matches an existing contact, should we covert it to a contact's name? If so, that should be different from phone number
- Contact suggestion works for email as well.
- In contact app, can user choose to send message to one's email?

Comment 19

[Joe Cheng [:jcheng] (please needinfo)]

1.3+ committed feature

Comment 20

[ayman maat :maat]

(In reply to Paul Yang [: pyang] from comment #17)
> Hi Ayman,
> just want to make sure if we cover these cases:
> - email matches an existing contact, should we covert it to a contact's name? If so, that should be different from phone number

We should maintain the same behaviour as is currently on the phone with regards to phone number.


> - Contact suggestion works for email as well.

Yes please. we need to include email addresses in the auto suggestions

> - In contact app, can user choose to send message to one's email?

…this is something i am trying to clarify now, will get back to you on this


leaving ni? to me

Comment 21

[Rick Waldron [:rwaldron]]

(In reply to ayman maat :maat from comment #20)
> (In reply to Paul Yang [: pyang] from comment #17)
> > Hi Ayman,
> > just want to make sure if we cover these cases:
> > - email matches an existing contact, should we covert it to a contact's name? If so, that should be different from phone number
> 
> We should maintain the same behaviour as is currently on the phone with
> regards to phone number.
> 
> 
> > - Contact suggestion works for email as well.
> 
> Yes please. we need to include email addresses in the auto suggestions
> 
> > - In contact app, can user choose to send message to one's email?
> 
> …this is something i am trying to clarify now, will get back to you on this
> 
> 
> leaving ni? to me

Just want to confirm the first two above. The third will require some effort from Contacts app, but will be no problem for Messages app

Comment 22

[ayman maat :maat]

(In reply to ayman maat :maat from comment #20)
> (In reply to Paul Yang [: pyang] from comment #17)
> > Hi Ayman,
> > just want to make sure if we cover these cases:
> >
> > - In contact app, can user choose to send message to one's email?
> 
> ...this is something i am trying to clarify now, will get back to you on this

I have confirmation that we will not be doing this for V1.3

Comment 23

[Rick Waldron [:rwaldron]]

(In reply to ayman maat :maat from comment #22)
> (In reply to ayman maat :maat from comment #20)
> > (In reply to Paul Yang [: pyang] from comment #17)
> > > Hi Ayman,
> > > just want to make sure if we cover these cases:
> > >
> > > - In contact app, can user choose to send message to one's email?
> > 
> > ...this is something i am trying to clarify now, will get back to you on this
> 
> I have confirmation that we will not be doing this for V1.3

That's a bummer! Is this moving on to 1.4?

Comment 24

[Rick Waldron [:rwaldron]]

Wilfred, 

It just occurred to me that Ayman's comment was only regarding that _one_ very specific case. Apologies for removing the tags.

Comment 25

[Rick Waldron [:rwaldron]]

Checked in twice on https://bugzilla.mozilla.org/show_bug.cgi?id=866938, no answer to first (11/15)

Comment 26

[Wayne Chang [:wchang]]

Not this is required by partner for Madai.

Comment 27

[Wayne Chang [:wchang]]

(In reply to Wayne Chang [:wchang] from comment #26)
> Not this is required by partner for Madai.

That was supposed to read "Note this is required by partner for Madai."

Comment 30

[Beatriz Rodríguez [:brg]]

Rick, are you still working in this feature?

Comment 31

[Rick Waldron [:rwaldron]]

(In reply to Beatriz Rodríguez [:brg] from comment #30)
> Rick, are you still working in this feature?

That was my plan, but there is nothing I (or anyone) can do until https://bugzilla.mozilla.org/show_bug.cgi?id=866938 is resolved.

Comment 32

[Beatriz Rodríguez [:brg]]

Editing the tittle trying to be more precisse.

Comment 33

[Rick Waldron [:rwaldron]]

(In reply to Beatriz Rodríguez [:brg] from comment #32)
> Editing the tittle trying to be more precisse.

Pinged that other ticket https://bugzilla.mozilla.org/show_bug.cgi?id=866938#c27

Comment 34

[Noemí Freire (:noemi)]

Triage: new feature so adding it to backlog

Comment 35

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

This is high priority feature from partner. Propose to be included in 1.5.

Comment 36

[Joe Cheng [:jcheng] (please needinfo)]

this is part of the partner contribution for madai

ni? cawang / omega for UX

Comment 37

[Omega Feng [:Omega] [:馮於懋]]

Attachment: [1.5 Messages] Send MMS to Email v1.0.pdf

Please see the UX spec for details.

Comment 38

[Omega Feng [:Omega] [:馮於懋]]

Hi Vicky, please help on the visual design, thanks!

Comment 39

[Victoria Gerchinhoren [:vicky]]

Attachment: Visual Mockup. Compose emailindicator hitstate

Comment 40

[Victoria Gerchinhoren [:vicky]]

Attachment: Visual Mockup. Compose email indicator

Comment 41

[Victoria Gerchinhoren [:vicky]]

Attachment: Visual Spec. Compose email indicator

Comment 42

[Victoria Gerchinhoren [:vicky]]

Attached Visual material, both mockups and specs to implement what's requiered in this bug. The asset for the envelope icon is to come.

Comment 43

[Victoria Gerchinhoren [:vicky]]

Attachment: Asset. Email indicator icon for recipients bubbles.

Comment 44

[Wayne Chang [:wchang]]

retiring madai flag
For mozilla we should have this by 1.5

tracking for madai with m+

Comment 45

[Joe Cheng [:jcheng] (please needinfo)]

triage: this is a feature for madai.
not plusing on a feature

Comment 46

[Carrie Wang [New account: [e-mail]]]

Attachment: [MADAI][Contact] Send to email address.pdf

Hey guys, for this feature, we need to add a message button in Contact details. I've uploaded the spec here. Please take a look. :)
For the visual design, I think we can adopt the same message button which is used for phone numbers. ni? Vicky to confirm it. Thanks!

Comment 47

[Victoria Gerchinhoren [:vicky]]

(In reply to Carrie Wang [:carrie] from comment #46)
> Created attachment 8404006 [details]
> [MADAI][Contact] Send to email address.pdf
> 
> Hey guys, for this feature, we need to add a message button in Contact
> details. I've uploaded the spec here. Please take a look. :)
> For the visual design, I think we can adopt the same message button which is
> used for phone numbers. ni? Vicky to confirm it. Thanks!

For the sake of consistency it hsould be just as it is with the numbers, so yes, agreeing woth you Carrie.

Thanks!

Comment 48

[Carrie Wang [New account: [e-mail]]]

Attachment: [MADAI][Contact] Send to email address v2.0.pdf

Hi I've found a mistake on flow title and uploaded a revised version here. Not a big deal. The flow is still the same. Thanks!

Comment 50

[Noemí Freire (:noemi)]

Hi Rick,

Bug 866938 has been already landed, are you still planning to work on this?. Thanks!

Comment 51

[Julien Wajsberg [:julienw]]

Rick has no time to work on this now, so removing NI and assignee.

Noemi, you must know that some work is already happening in bug 974867.

Comment 53

[Julien Wajsberg [:julienw]]

Salva, is there a reason you assigned yourself on this bug? This is mostly a meta bug :)

Comment 54

[Maria Angeles Oteo (:oteo)]

(In reply to Julien Wajsberg [:julienw] from comment #53)
> Salva, is there a reason you assigned yourself on this bug? This is mostly a
> meta bug :)

You are right, :)

One thing, Julien, I asked Salva to talk to you about this feature so he can help with the development as it's taking longer and it's an important certification waiver.

As you know the feature status better that us, let us know where are the best candidate bugs to start working on. Thanks!!

Comment 55

[Salvador de la Puente González [:salva]]

I knew it. I was assigning to me as reminder. I'm gonna review the bug's blockers, ask for some permissions and ask to you, Julien, of course. ;)

Comment 56

[Julien Wajsberg [:julienw]]

The main thing that's left to enable the feature is the blocking bug 1026384. Bug 997547 and bug 1030672 are nice to have (I think).

The blocking bug is already being worked on by a partner currently; it's really no big deal, but the partner takes some time to do it. You can discuss with him if you want to take over the bug.

Bug 997547 is awaiting a review from Francisco.
Bug 1030672: we need to assess if it's needed.
There are other dependencies in the restricted bug 974874, but they are mostly nice to have.

Comment 57

[Julien Wajsberg [:julienw]]

Hubert, can I please access bug 1071495 ?

Comment 58

[Hubert Lu[:hlu] <[e-mail]>]

Hi Julien,
   Add you in cc list. Pleas check it.

Comment 59

[Julien Wajsberg [:julienw]]

Hubert, I'm still unauthorized to access the bug. Maybe it's not important, but because you added it here I thought it could be useful that I have access to it.

Comment 60

[Hubert Lu[:hlu] <[e-mail]>]

Hi Julien,
    Has confirmed that your email account is added in bug 1071495 cc list. Please try to access it again. Thank you.

Comment 61

[田旻]

[Blocking Requested - why for this release]:

Comment 62

[Wesly Huang (EPM)]

Hi Julien, I'm trying to digest the history of comments and my understanding in short is, this feature is completed already in FFOS 1.5 after bug 1026384 is done (based on comment#56), right? (Not considering other nice to have bugs like Bug 997547 and bug 1030672)

Thanks in advance.

Comment 63

[Wilfred Mathanaraj [:WDM]]

no dont think this is done - MMS to email address is still pending between madai and other partner interest to complete the work.

Comment 64

[田旻]

(In reply to 田旻 from comment #61)
> [Blocking Requested - why for this release]:

Dears, 

Can this bug be fixed on v2.0?

BRs,
TIAN Min

Comment 65

[Wilfred Mathanaraj [:WDM]]

I dont think this feature will happen in 2.0 - I also dont think it will happen in 2.1. The best possible solution is this happening in 2.2.

Comment 66

[Beatriz Rodríguez [:brg]]

Thanks Wilfred! We hope so :)

Comment 67

[Wilfred Mathanaraj [:WDM]]

before we say this is planned for 2.2 - I want to make sure we understand that there are a lot of IFs and BUTs. so lets not take what i said above as that we are mandating this is coming in 2.2.

But there is a good chance for us to bring this into 2.2 if there is the right amount of contribution and testing.

Comment 68

[Wesly Huang (EPM)]

Thanks Wilfred's information sharing! Then at least it's clear we won't fix it in 2.0 & 2.1, so plan to de-nominate it from 2.0 in coming Triage.

Comment 69

[Julien Wajsberg [:julienw]]

All bugs except bug 1079257 are necessary. They're all quite small and easy to do, except bug 1078384 that needs some CSS change and where we haven't completely finished to settle on a spec.

It's all a matter of focussing. Right now the priority is not on this feature so we don't move quickly on this. The work is done by partners and it looks like that even partners are busy on other things.

Comment 70

[Maria Angeles Oteo (:oteo)]

(In reply to Julien Wajsberg [:julienw] from comment #69)
> It's all a matter of focussing. Right now the priority is not on this
> feature so we don't move quickly on this. The work is done by partners and
> it looks like that even partners are busy on other things

Salva will continue working on them next week (this week is on PTO) if no more Cost Control issues raise.

Comment 71

[Wesly Huang (EPM)]

Thanks for Julien and Maria's information! So in short we can say "targeting 2.2" but clear that won't fix it in 2.0/2.1. Still a de-nominate candidate in coming 2.0 Triage.

Comment 72

[Maria Angeles Oteo (:oteo)]

(In reply to Wesly Huang from comment #71)
> Thanks for Julien and Maria's information! So in short we can say "targeting
> 2.2" but clear that won't fix it in 2.0/2.1. Still a de-nominate candidate
> in coming 2.0 Triage.

agreed, unfortunately, too late for those versions

Comment 73

[Wesly Huang (EPM)]

[Triage] by 2.0 design.

Comment 75

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

[Tracking Requested - why for this release]:

Comment 76

[Julien Wajsberg [:julienw]]

This is now mostly working. 2 bugs are left, but that do not impair the functionality, so I'm closing this meta bug now.