893766 - SMS Web Activity call from 3rd party web apps wont transfer phone number

Status: RESOLVED WORKSFORME

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

Description

[zoltanf]

Attachment: Screenshot_2013-07-15-15_55_52.png

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/28.0.1500.71 Safari/537.36

Steps to reproduce:

Run this example code in Gaia, in a 3dt party packaged web app:

            var sms = new MozActivity({
                name: "new", // Possible compose-sms in future versions
                data: {
                    type: "websms/sms",
                    number: "[phone]",
                    body: "my test sms text"
                }
            });


Actual results:

The New SMS activity is open but the phone number is not filled in (only the SMS body). This happens in OS v1.2 preview (test on Geeksphone Peak)


Expected results:

New SMS activity is shown with filled data: phone number and sms message body.

Comment 1

[Jason Smith [:jsmith]]

Confirmed. I'm seeing this on v1-train as well. Got a partner compliant as well from partner engineering.

Comment 2

[Rick Waldron [:rwaldron]]

I believe this was fixed here: https://github.com/mozilla-b2g/gaia/commit/aaafec3c79e94041db7d22d3490f8d4916faeb3a

Can you confirm with gaia@master?

Comment 3

[Anthony Ricaud (:rik)]

That would explain why I can't reproduce today.

Marking as worksforme because of that.

Zoltan, feel free to reopen if you can still see this with a build from today.

Comment 4

[zoltanf]

I don't know if this was fixed in v1.1, currently I have no way to test.

On v1.2 pre-release it is working fine with a nightly build for Peak, 16-07-2013:
nightly-images-peak-2013-07-16.Gecko-d38f55b.Gaia-759f4b2

Is it fixed also in v1.1?

Comment 5

[Anthony Ricaud (:rik)]

From my testing, yes it is. And the commit Rick mentioned has been merged in 1.1.