975259 - [Gaia] Add protocol and roaming protocol to APN settings.

Status: RESOLVED INVALID

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[Ken Chang[:kenkai]]

We may need to add protocol and roaming protocol options into APN setting.

Comment 1

[José Antonio Olivera Ortega [:jaoo]]

Let me define the scope of this bug and what we are going to do here. We will take care of the UI changes in the APN panels for adding protocol and roaming protocol fields in the APN form.

Comment 2

[Jessica Jong [:jessica] (inactive)]

(In reply to José Antonio Olivera Ortega [:jaoo] from comment #1)
> Let me define the scope of this bug and what we are going to do here. We
> will take care of the UI changes in the APN panels for adding protocol and
> roaming protocol fields in the APN form.

Thank you José for defining the scope of this bug. What about the part for reading protocol/roaming protocol from the apn.json database? Will this be handled in this bug as well or is there another bug for this?

Comment 3

[José Antonio Olivera Ortega [:jaoo]]

(In reply to Jessica Jong [:jjong] [:jessica] from comment #2)
> Thank you José for defining the scope of this bug. What about the part for
> reading protocol/roaming protocol from the apn.json database? Will this be
> handled in this bug as well or is there another bug for this?

`protocol` and `roaming_protocol` properties are already present for some APN in the apn.json data base. Those properties are already part of the APN the RIL plumbing receives on boot and will be passed as well for the APNs selected in the setting app as well when this bug lands.

Comment 4

[José Antonio Olivera Ortega [:jaoo]]

I wont be able to finish this bug for 2/28. Leaving the bug so anyone else could take and work on it.

Comment 5

[José Antonio Olivera Ortega [:jaoo]]

As this bug is likely to add a couple of new fields in the APN editor panel being added in bug 969298, I'll take care of it in that bug since bug 969298 is redefining the whole APN UI.

I'll resolve this bug as INVALID (please change that if it is wrong).

Comment 6

[José Antonio Olivera Ortega [:jaoo]]

The work here has been added to the patch for bug 969298.