[Gaia][Settings] Use network info in state change event instead of read current network info.
RESOLVED
FIXED
in 2.1 S7 (24Oct)
Status
People
(Reporter: chucklee, Assigned: chucklee)
Tracking
Firefox Tracking Flags
(Not tracked)
Details
Attachments
(1 attachment)
|
46 bytes,
text/x-github-pull-request
|
eragonj
:
review+
|
Details | Review | Splinter Review |
Most state change event already contains corresponding network info, and after bug 1050678, we expect have network info in every state change event. We could use these information while handling state change event to improve user experience.
|
(Assignee) | |
Comment 1•5 years ago
|
||
Use wifi network information from event if available.
Attachment #8475037 -
Flags: feedback?(ejchen)
Comment on attachment 8475037 [details] [review] Pull Request Chuck, just left some comments on github, please give it a check !
Attachment #8475037 -
Flags: feedback?(ejchen)
|
|
(Assignee) | |
Updated•5 years ago
|
Attachment #8475037 -
Flags: review?(ejchen)
Chuck, this patch should be landed after bug 1050678, right ? Because currently Gecko is not reflecting with this change, I'll keep the r? on my side and test after that got landed. WDYK !? :)
Flags: needinfo?(chulee)
Basically they are good to me :)
Comment on attachment 8475037 [details] [review] Pull Request Sorry for the late reply, Chucklee. This patch looks ok to me and we just have too add one unit test for _updateNetworkStatus and update nits about indentations. Thanks !
Attachment #8475037 -
Flags: review?(ejchen)
|
|
(Assignee) | |
Comment 7•5 years ago
|
||
Comment on attachment 8475037 [details] [review] Pull Request Fix nits and add unit test based existing test case.
Attachment #8475037 -
Flags: review?(ejchen)
Comment on attachment 8475037 [details] [review] Pull Request Thanks Chucklee ! r+ :)
Attachment #8475037 -
Flags: review?(ejchen) → review+
|
|
(Assignee) | |
Updated•5 years ago
|
Keywords: checkin-needed
|
||
Comment 9•5 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/3f3ca2e6f8c2e64e5704d879ed8c7c79e6ce4dbd
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S7 (24Oct)
You need to log in
before you can comment on or make changes to this bug.
Description
•