Closed
Bug 923443
Opened 11 years ago
Closed 11 years ago
[User Story] Single variant: Pre-polulate some WiFi SSID by SIM
Categories
(Firefox OS Graveyard :: Gaia::System, defect, P1)
Tracking
(feature-b2g:2.0, tracking-b2g:backlog)
RESOLVED
FIXED
1.4 S5 (11apr)
People
(Reporter: sonmarce, Assigned: macajc)
References
Details
(Keywords: feature, Whiteboard: [systemsfe][ucid:System12, systemsfe][p=5][leave open])
User Story
As an Operator I want to have a number of predefined stored WiFi SSID per country, so that users can connect to them automatically Acceptance criteria: * WiFi SSID will be configured per each MCC/MNC after inserting a new SIM card and entering PIN code for the first time * All required parameters will be stored depending on the kind of WiFi SSID * In case there are already some WiFi SSID previously configured by the user, they will be preserved, just adding the new ones * Replacing or removing SIM card later on will imply no change in WiFi SSID configuration * After a factory reset, WiFi SSID configuration will follow same process as a brand new device
Attachments
(1 file, 3 obsolete files)
(deleted),
patch
|
vchang
:
review+
|
Details | Diff | Splinter Review |
As an Operator I want to have a number of predefined stored WiFi SSID per country, so that users can connect to them automatically
Acceptance criteria:
* WiFi SSID will be configured per each MCC/MNC after inserting a new SIM card
* To complete connection, user will follow currently available process: authentication by password, captive portal, ...
Reporter | ||
Updated•11 years ago
|
Reporter | ||
Updated•11 years ago
|
Priority: -- → P1
Updated•11 years ago
|
Whiteboard: [systemsfe] → [ucid:System12, systemsfe]
Updated•11 years ago
|
QA Contact: rafael.marquez
Updated•11 years ago
|
No longer blocks: 1.3-systems-fe
Reporter | ||
Updated•11 years ago
|
Blocks: 1.4-systems-fe
Reporter | ||
Updated•11 years ago
|
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
Reporter | ||
Updated•11 years ago
|
Whiteboard: [ucid:System12, systemsfe] → [systemsfe][ucid:System12, systemsfe]
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → cjc
Reporter | ||
Comment 1•11 years ago
|
||
We have updated acceptance criteria to be as follows:
* WiFi SSID will be configured per each MCC/MNC after inserting a new SIM card and entering PIN code for the first time
* All required parameters will be stored depending on the kind of WiFi SSID
* In case there are already some WiFi SSID previously configured by the user, they will be preserved, just adding the new ones
* Replacing or removing SIM card later on will imply no change in WiFi SSID configuration
* After a factory reset, WiFi SSID configuration will follow same process as a brand new device
Also starting to use the new User Story field in Bugzilla.
Reporter | ||
Updated•11 years ago
|
User Story: (updated)
Updated•11 years ago
|
Target Milestone: 1.3 C2/1.4 S2(17jan) → 1.3 C3/1.4 S3(31jan)
Updated•11 years ago
|
blocking-b2g: --- → backlog
Assignee | ||
Comment 2•11 years ago
|
||
The user history requires Pre-populating some WiFi SSID (and the set of SSIDs to prepopulate depend on the inserted SIM) when the phone is powered up for first time (and posiblemente said wifi networks aren't visible at the phone actual location). So we need the ability to associate a Wifi without trying to connect to it. The objective of this patch is allow to do it.
Attachment #8370600 -
Flags: review?(vchang)
Assignee | ||
Comment 3•11 years ago
|
||
Attachment #8370602 -
Flags: review?(francisco.jordano)
Updated•11 years ago
|
Target Milestone: 1.3 C3/1.4 S3(31jan) → 1.4 S1 (14feb)
Comment 4•11 years ago
|
||
Comment on attachment 8370602 [details]
Proposed patch v1 - gaia
Left some comments on github, non blocking, mainly style.
Thanks Carmen!
Attachment #8370602 -
Flags: review?(francisco.jordano) → review+
Updated•11 years ago
|
Flags: in-moztrap?(rafael.marquez)
Updated•11 years ago
|
Whiteboard: [systemsfe][ucid:System12, systemsfe] → [systemsfe][ucid:System12, systemsfe][p=5]
Comment 5•11 years ago
|
||
Comment on attachment 8370600 [details] [diff] [review]
Proposed patch - v1
Review of attachment 8370600 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/wifi/WifiWorker.js
@@ +2640,5 @@
> + let privnet = network;
> + let dontConnect = privnet.dontConnect;
> + delete privnet.dontConnect;
> +
> + if (!WifiManager.enabled && !dontConnect) {
Why should we check dontConnect here ?
@@ +2650,5 @@
> function networkReady() {
> // saveConfig now before we disable most of the other networks.
> function selectAndConnect() {
> + if (dontConnect) {
> + self._sendMessage(message, true, "Wifi has been recorded", msg);
Can we check this flag out of selectAndConnect() function ? It makes more sense for the function name.
Attachment #8370600 -
Flags: review?(vchang)
Assignee | ||
Comment 6•11 years ago
|
||
This version includes the previous review comments
Attachment #8373439 -
Flags: review?(vchang)
Comment 7•11 years ago
|
||
Comment on attachment 8373439 [details] [diff] [review]
bug923443_v2.patch
Review of attachment 8373439 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/wifi/WifiWorker.js
@@ +2664,5 @@
> });
> }
>
> if (self._highestPriority >= MAX_PRIORITY)
> self._reprioritizeNetworks(selectAndConnect);
Do we also need to check dontConnect flag here ? The other question is do these default APs have lowest priority than APs chose by user ?
Assignee | ||
Comment 8•11 years ago
|
||
You're right, when I took the condition check out of selectAndConnect I forgot about the other case. Fixed that.
As for the priority, currently they're just added with the default priority. I don't think that's actually part of the user story.
In any case, I don't think that on a normal use one of the default networks will collide with a user defined one (since mostly the default networks will correspond to operator provided public access points).
Attachment #8370600 -
Attachment is obsolete: true
Attachment #8373439 -
Attachment is obsolete: true
Attachment #8373439 -
Flags: review?(vchang)
Attachment #8376546 -
Flags: review?(vchang)
Comment 9•11 years ago
|
||
Comment on attachment 8376546 [details] [diff] [review]
Proposed patch, v3
Review of attachment 8376546 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good, thank you.
Attachment #8376546 -
Flags: review?(vchang) → review+
Assignee | ||
Comment 10•11 years ago
|
||
Assignee | ||
Comment 11•11 years ago
|
||
The try run looks good, requesting checkin. Please do not land the Gaia patch until the Gecko patch has landed.
Keywords: checkin-needed
Comment 12•11 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/5ba2e35f4d48
Just put checkin-needed back on this bug when you're ready for the Gaia PR to merge. :)
Keywords: checkin-needed
Whiteboard: [systemsfe][ucid:System12, systemsfe][p=5] → [systemsfe][ucid:System12, systemsfe][p=5][leave open]
Updated•11 years ago
|
Target Milestone: 1.4 S1 (14feb) → 1.4 S2 (28feb)
Updated•11 years ago
|
Target Milestone: 1.4 S2 (28feb) → 1.4 S3 (14mar)
Comment 14•11 years ago
|
||
Is this 1.5 now?
Assignee | ||
Comment 15•11 years ago
|
||
Since we're moving all the Single Variant customizations into a stand alone app, I think it's better to just not land the Gaia part (and close this bug as it is). If you think different, I can fix the conflicts and merge the Gaia patch.
Please, see bugs 978785 and 980301.
Assignee | ||
Comment 16•11 years ago
|
||
Please, see bug 978785 and bug 980301.
Reporter | ||
Updated•11 years ago
|
Comment 17•11 years ago
|
||
moving to next sprint since this won't land till after 3/17 FC
Target Milestone: 1.4 S3 (14mar) → 1.4 S4 (28mar)
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Updated•11 years ago
|
Attachment #8370602 -
Attachment is obsolete: true
Reporter | ||
Updated•11 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Comment 18•11 years ago
|
||
Closed by mistake, still a dependent bug pending to be finished
Reporter | ||
Updated•11 years ago
|
Target Milestone: 1.4 S4 (28mar) → 1.4 S5 (11apr)
Reporter | ||
Comment 19•11 years ago
|
||
Implementation completed in bug 978785
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
feature-b2g: --- → 2.0
Updated•10 years ago
|
blocking-b2g: backlog → ---
tracking-b2g:
--- → backlog
You need to log in
before you can comment on or make changes to this bug.
Description
•