Closed
Bug 904984
Opened 12 years ago
Closed 12 years ago
[gaia] Need to add APN settings of CDMA carriers to the database for QA testing
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: arthurcc, Assigned: arthurcc)
References
Details
Attachments
(1 file)
Need to add APN settings of CDMA carriers to the database. For example Sprint in the US and in Asia Pacific Telecom in Taiwan.
Assignee | ||
Comment 1•12 years ago
|
||
Unblock 906596 since this is not a gaia bug.
No longer blocks: 906596
Assignee | ||
Comment 2•12 years ago
|
||
The added settings was found on the internet. I am not able to test these settings because MMS is not supported in v1.2 and data connection is still not working for now [1].
[1] Bug 906587 - [Wasabi] Reference phone cannot camp on EVDO network.
Attachment #792713 -
Flags: feedback?(josea.olivera)
Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
Switching to Kaze since Antonio is on PTO.
Attachment #792713 -
Flags: feedback?(josea.olivera) → review?(kaze)
Hi Arthur,
Actually only EVDO network is not working, you can still using 1x data call with APTG UIM card on wasabi. FYI.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → arthur.chen
Comment 5•12 years ago
|
||
Some feedback here with Kaze's permission. We have never added anything to the database (e.g. apns_conf.xml) files used to build our apn.json database so I recommend to continue doing the same for APN settings for CDMA carriers. Arthur, you are adding APN sets in those files. I'd recommend to add a new database file for CDMA carriers as CyanogenMod project does (see [1]).
[1] https://github.com/CyanogenMod/android_vendor_cm/blob/ics/prebuilt/common/etc/apns-conf-cdma.xml
Assignee | ||
Comment 6•12 years ago
|
||
Jose, thanks! Patch was updated based on the comments.
Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
Jose, are you able to review this? This could save lots of time of QA inputing the APN settings every time they reflash builds.
Attachment #792713 -
Flags: review?(kaze) → review?(josea.olivera)
Assignee | ||
Comment 8•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
Kaze, I had a wrong impression of that Jose is able to review the patch. Could you help review the change? Thanks!
Attachment #792713 -
Flags: review?(josea.olivera) → review?(kaze)
Comment 9•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
FTR, Jose is the most qualified developer to review APN patches.
r=me, but pinging Jose in case he has something to say before we merge this.
Attachment #792713 -
Flags: review?(kaze)
Attachment #792713 -
Flags: review+
Attachment #792713 -
Flags: feedback?(josea.olivera)
Comment 10•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
Just noticed the APN database for the CDMA carriers in the PR is not the one I commented about in comment #5. We use open source APN databases and IMHO we should do the same for APN settings for CDMA carriers. If the CDMA carrier we want to test with is not in the APN database from comment #5 the best we can do is to add those APN settings to the apns_conf_local.xml database.
Attachment #792713 -
Flags: feedback?(josea.olivera) → feedback-
Assignee | ||
Comment 11•12 years ago
|
||
It seems that I misunderstood comment #5. I thought we need to create our own database for cdma. I'll update the patch to move the newly added APN settings to apns_conf_local.xml.
Assignee | ||
Comment 12•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
Antonio, I've updated the patch. Would you mind take a look at it again? Thanks!
Attachment #792713 -
Flags: feedback- → feedback?(josea.olivera)
Comment 13•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
Arthur, I left a comment in github. would you mind to take a look please? Thanks.
Attachment #792713 -
Flags: feedback?(josea.olivera)
Assignee | ||
Comment 14•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
I left a comment in github, could you help check it? Thanks!
Attachment #792713 -
Flags: feedback?(josea.olivera)
Comment 15•12 years ago
|
||
Comment on attachment 792713 [details]
link to https://github.com/mozilla-b2g/gaia/pull/11627
Sorry for the lag. Looks good to me. Try to rebuild the apn.json database again before landing this patch. Some changes has been landed around the apn.json builder script and we must be sure everything's ok before landing a change in the database. Thanks!
Attachment #792713 -
Flags: feedback?(josea.olivera) → feedback+
Assignee | ||
Comment 16•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 17•12 years ago
|
||
Verified on Wasabi master build but still not having correct APN programmed first.
Gaia: b2f3fc404a0e7c577c49093f8300ab7e73154de9
Gecko: 32c8237212fcf578be7c0e00bfeb9c0ba06c4816
BuildID 20131015121031
Version 27.0a1
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 18•12 years ago
|
||
(In reply to Enpei from comment #17)
> Verified on Wasabi master build but still not having correct APN programmed
> first.
>
> Gaia: b2f3fc404a0e7c577c49093f8300ab7e73154de9
> Gecko: 32c8237212fcf578be7c0e00bfeb9c0ba06c4816
> BuildID 20131015121031
> Version 27.0a1
You should have the new APN added for the SIM card with mcc="466" mnc="05" values. Are you using that SIM card?
Flags: needinfo?(echu)
Comment 19•12 years ago
|
||
Yes, I've passed device and SIM card(APTG account) to Edgar as well who is checking the bug now. Same card on same model device but with Android OS can program APN automatically.
Flags: needinfo?(echu)
Assignee | ||
Comment 20•12 years ago
|
||
FYI in this case gecko returns incorrect mcc and mnc values.
Comment 21•12 years ago
|
||
Enpei - Can you file a followup bug here?
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Flags: needinfo?(echu)
Updated•12 years ago
|
Flags: needinfo?(echu)
Assignee | ||
Comment 22•12 years ago
|
||
Revert the patch as there is a typo in it.
master: a82759418d6d56f276d1e917fe95eb268e0470b4
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 23•12 years ago
|
||
Resolved.
master: 62a84888f1c10556af8fbbd50e126bb1afe96276
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•