Closed Bug 890826 Opened 6 years ago Closed 6 years ago

[User story][CDMA] Display CDMA information

Categories

(Firefox OS Graveyard :: RIL, defect, P1)

x86
macOS
defect

Tracking

(blocking-b2g:koi+, firefox26 fixed)

RESOLVED FIXED
blocking-b2g koi+
Tracking Status
firefox26 --- fixed

People

(Reporter: khu, Unassigned)

References

Details

(Whiteboard: [ucid:CDMA12, FT:RIL, KOI:P1])

Attachments

(1 file)

User story: 

As a user, I need an UI support to display CDMA info record as defined in C.S0005 section 3.7.5.1 and extended Display Info Rec as defined in C.S0005 section 3.7.5.16

Acceptance criteria: 

User should be able to display CDMA info record as defined in C.S0005 section 3.7.5.1 and extended Display Info Rec as defined in C.S0005 section 3.7.5.16
blocking-b2g: --- → koi+
Depends on: 882985
Depends on: 891755
Flags: in-moztrap?
QA Contact: echu
Hi Sandip,

Could you provide more detail about how this should be seen for end user? How to trigger it? What are contents shown on the UI?
Flags: needinfo?(skamat)
@echu, the CDMA display information record needs to be shown as a dialog box just like how a dialog box is shown for presidential alerts by cell broadcast feature in FFOS.

The CDMA signal info record could either be played by dialer using the tone generator that it already has or by RIL. 

Rest of the info records are not supported even by Android at this time so I am not sure if they would need to be for FFOS. Sandip, could you please help find out if the rest of the info records need to be supported?
(In reply to Anshul from comment #2)
> @echu, the CDMA display information record needs to be shown as a dialog box
> just like how a dialog box is shown for presidential alerts by cell
> broadcast feature in FFOS.
> 
> The CDMA signal info record could either be played by dialer using the tone
> generator that it already has or by RIL. 
> 
> Rest of the info records are not supported even by Android at this time so I
> am not sure if they would need to be for FFOS. Sandip, could you please help
> find out if the rest of the info records need to be supported?

At this time (till partners confirm) we will leave it to be consistent with Android implementations.
Flags: needinfo?(skamat)
(In reply to Anshul from comment #2)
> @echu, the CDMA display information record needs to be shown as a dialog box
> just like how a dialog box is shown for presidential alerts by cell
> broadcast feature in FFOS.
How to trigger this CB message? Do you have example screenshot?

> The CDMA signal info record could either be played by dialer using the tone
> generator that it already has or by RIL. 
Can you check is the attached screenshot is what you are asking for? It's triggered by dialer on android phone after dialing customized code(ex:##1234). It will show the signal of 1x, EVDO and GPS.
Attached image Screenshot.png
Priority: -- → P1
@echu, I don't think there is a specification on how to display the CDMA info record at least nothing that I could find in C.S0005 section 3.7.5.

I think all we need is a way to display the information with in the info record to the user. How it is done can be Moz specific, I am just telling based on how Android displays the information.
Flags: in-moztrap? → in-moztrap?(echu)
Whiteboard: [ucid:CDMA12] → [ucid:CDMA12], [FT:RIL]
Whiteboard: [ucid:CDMA12], [FT:RIL] → [ucid:CDMA12, FT:RIL, KOI:P1]
Hi, Anshul, can you provide some examples for us to know what to display? We still don
't know what to display yet.
Flags: needinfo?(anshulj)
@khu, let me try to run this feature on Android and get some snapshot. I am not available for most part of this week so please expect a reply by end of this week.
Flags: needinfo?(anshulj)
(In reply to Anshul from comment #8)
> @khu, let me try to run this feature on Android and get some snapshot. I am
> not available for most part of this week so please expect a reply by end of
> this week.
Anshul, do you have any update for this snapshot?
Flags: needinfo?(anshulj)
Sorry Ken, I didn't. Let me try to get some test done tomorrow and get you the snapshots. Thanks for the reminder.
Flags: needinfo?(anshulj)
Sandip, we don't have the real environment to do this test. I am thinking if we can only provide webAPI for this bug. And leave partner to finish the Gaia part. The another way is that we do the Gaia part when we have a real partner and know our partner's exactly requirement. Does it make sense to you?
Flags: needinfo?(skamat)
Ken, you guys can finish the gaia part and I can help test it internally. I am still working with the test teams to get me a setup to run this test.
(In reply to Anshul from comment #12)
> Ken, you guys can finish the gaia part and I can help test it internally. I
> am still working with the test teams to get me a setup to run this test.
We need the snapshots to know how to implement this in Gaia. Thanks.
Flags: needinfo?(skamat)
Ken, I am still working with our internal test teams to try to reproduce this scenario. From the code scan it seems like the display information should be shown just like Cell broadcast message, snapshot at https://bug848372.bugzilla.mozilla.org/attachment.cgi?id=756260.
https://bugzilla.mozilla.org/show_bug.cgi?id=891755#c9

Since we don't have a detail information to show the information record, I suggest to remove the Gaia implementation from this user story.  We can start Gaia's implementation after having detail information.
Flags: needinfo?(skamat)
This could just be vanilla dialog box containing the network text with an OK/dismiss button at the bottom iirc.  Lets implement that in Gaia for now and if we need to tune the UX later then that's fine.
Flags: needinfo?(anshulj)
Ken, I agree with Micahel that we can just display a dialog box with text message from the network. There is no spec I could find that says how to display the message so a generic message for now would do.
Flags: needinfo?(anshulj)
Vicamo, if we are based on the information of comment 16 and comment 17, is it fine for you to keep reviewing bug 882985?
Flags: needinfo?(vyang)
(In reply to Ken Chang from comment #18)
> Vicamo, if we are based on the information of comment 16 and comment 17, is
> it fine for you to keep reviewing bug 882985?

Ok, will do.
Flags: needinfo?(vyang)
Flags: needinfo?(skamat)
Not knowing how to setup the test environment at current stage, no test case is ready for now.
Flags: in-moztrap?(echu) → in-moztrap-
This should be done in this week.
Component: General → RIL
Add dependent bugs are close, close this user story.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.