Closed
Bug 1026098
Opened 10 years ago
Closed 10 years ago
[User Story] Skip time and timezone setting when network time available
Categories
(Firefox OS Graveyard :: Gaia::First Time Experience, defect)
Tracking
(feature-b2g:2.1, b2g-v2.1 verified, b2g-v2.2 verified)
People
(Reporter: pdol, Assigned: tedders1)
References
Details
(Keywords: feature, Whiteboard: [ucid:System211], [systemsfe][tako][2.1-feature-qa+])
User Story
As a user I don't want to have to set the time and timezone on my new device if it can be provided by the network so that I can use my device quicker. Acceptance Criteria: 1. During FTE if the time/timezone is successfully determine from the network, the time/timezone selection is not shown. 2. During FTE if the time/timezone cannot be determined from the network, the time/timezone selection is shown.
Attachments
(1 file, 1 obsolete file)
No description provided.
Reporter | ||
Updated•10 years ago
|
User Story: (updated)
Comment 2•10 years ago
|
||
Not an expert on the matter, as all the developing of the timezone selection was made by Kaze, but after looking at the code I'd say it would be easy.
Right now we check for the MCC/MNC combination to guess the timezone, and if we don't have it, fallback to America/New York. We could just show the screen if at the end of the process we end up with the default value.
I'd need to do some tests to check if the approach works, as we've had a lot of problems with detection on the past.
Flags: needinfo?(fernando.campo)
Updated•10 years ago
|
Comment 3•10 years ago
|
||
Ted is going to take this one.
Assignee: nobody → tclancy
blocking-b2g: backlog → 2.1?
Whiteboard: [ucid:System211], [ft:systemsfe] → [ucid:System211], [systemsfe]
Target Milestone: --- → 2.1 S1 (1aug)
Updated•10 years ago
|
blocking-b2g: 2.1? → ---
Whiteboard: [ucid:System211], [systemsfe] → [ucid:System211], [systemsfe][tako]
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8463605 -
Flags: review?(fernando.campo)
Comment 5•10 years ago
|
||
Comment on attachment 8463605 [details]
Bug 1026098 - Skip time and timezone setting when network time available.
I left some comments on the PR, to move the logic to a different place, as we have a function 'skipStep' that we can use instead, so we do all the checks for steps in the same place.
Also, we need tests on this. Please flag me for review when done.
And thanks for taking care of it.
Attachment #8463605 -
Flags: review?(fernando.campo)
Assignee | ||
Comment 6•10 years ago
|
||
I added unit tests, and fixed the old unit tests too.
Attachment #8463605 -
Attachment is obsolete: true
Attachment #8465057 -
Flags: review?(fernando.campo)
Comment 7•10 years ago
|
||
Comment on attachment 8465057 [details]
Skip time and timezone setting when network time available.
Left some comments, and a proposal for change. Tell me what do you think about it.
Looking good so far :)
Attachment #8465057 -
Flags: review?(fernando.campo) → feedback+
Assignee | ||
Updated•10 years ago
|
Attachment #8465057 -
Flags: review?(fernando.campo)
Assignee | ||
Comment 8•10 years ago
|
||
Thanks, Fernando. I've made the changes you suggested.
Comment 9•10 years ago
|
||
Comment on attachment 8465057 [details]
Skip time and timezone setting when network time available.
Looks great, thanks a lot!
Only comment left is to change 'SOURCE' variable to lowercase, linter is failing and patch doesn't work with that. I tested it after changing it, and works like a charm.
Merge whenever TBPL is happy :)
Attachment #8465057 -
Flags: review?(fernando.campo) → review+
Assignee | ||
Comment 10•10 years ago
|
||
D'oh! My bad.
Thanks, Fernando :-)
Assignee | ||
Comment 11•10 years ago
|
||
Successful try run: https://tbpl.mozilla.org/?rev=440de7f061bc1035f22914053e07b2f2f471c7e1&tree=Gaia-Try
Keywords: checkin-needed
Comment 12•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-b2g-v2.1:
--- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: 2.1 S1 (1aug) → 2.1 S2 (15aug)
Updated•10 years ago
|
Attachment #8465057 -
Flags: ui-review?(jsavory)
Updated•10 years ago
|
QA Whiteboard: [2.1-feature-qa+]
Updated•10 years ago
|
Flags: in-moztrap?(jsmith)
Updated•10 years ago
|
QA Contact: jsmith
Updated•10 years ago
|
QA Whiteboard: [2.1-feature-qa+]
Whiteboard: [ucid:System211], [systemsfe][tako] → [ucid:System211], [systemsfe][tako][2.1-feature-qa+]
Comment 13•10 years ago
|
||
Just tried with an up to date build (and gaia commit 254ecd807e539eee0b9a0af409b2f30cbb687fa5), and I can see the Date&Time screen after getting connected to the network. Flame, O2 UK SIM.
https://www.youtube.com/watch?v=y8Dz9jeZjEo
asking for qa to double check, and reopen or create new bug if needed.
Keywords: qawanted
Comment 14•10 years ago
|
||
(In reply to Fernando Campo (:fcampo) from comment #13)
> Just tried with an up to date build (and gaia commit
> 254ecd807e539eee0b9a0af409b2f30cbb687fa5), and I can see the Date&Time
> screen after getting connected to the network. Flame, O2 UK SIM.
> https://www.youtube.com/watch?v=y8Dz9jeZjEo
>
> asking for qa to double check, and reopen or create new bug if needed.
The discussion on bug 1049149 summarizes we are seeing - we do see the timezone screen getting skipped when the network is detected, but the timezone isn't be set correctly. Does that answer your question?
If you are reproducing this with your own SIM, then I think we should open a separate bug for that.
Flags: needinfo?(fernando.campo)
Comment 15•10 years ago
|
||
Thanks Jason, I'll open a new bug then, as I am still seeing the screen.
Flags: needinfo?(fernando.campo)
Keywords: qawanted
Updated•10 years ago
|
Flags: in-moztrap?(jsmith) → in-moztrap?(mozbugs.retornam)
QA Contact: jsmith → mozbugs.retornam
Updated•10 years ago
|
Flags: in-moztrap?(mozbugs.retornam) → in-moztrap?(gchang)
QA Contact: mozbugs.retornam → gchang
Comment 16•10 years ago
|
||
Add the test cases.
https://moztrap.mozilla.org/manage/cases/?filter-suite=750
Flags: in-moztrap?(gchang) → in-moztrap+
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
status-firefox34:
--- → verified
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Comment 17•10 years ago
|
||
This issue is verified fixed on the flame 2.1 (319mb).
Environmental Variables:
Device: Flame Master
Build ID: 20140903000204
Gaia: fbb297c39aab5f17b179533d2a9a6c5166b2c197
Gecko: fb5e796da813
Version: 34.0a2 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
During FTE, tap 'next' on cellular data screen, the time/timezone selection will not be shown. User can use the device quicker by not setting the time and timezone.
During FTE, 'Enable Data' on cellular data screen, the time/timezone selection is shown.
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Comment 18•10 years ago
|
||
After further investigation, The time/timezone selection is always skipped.
When choosing to not enable data or wifi, the time/timezone selection is still skipped.
The only way to display the time/timezone selection screen is to remove the SIM card before beginning the FTE.
Comment 19•10 years ago
|
||
(In reply to Seth Miko [:SethM] from comment #18)
> After further investigation, The time/timezone selection is always skipped.
> When choosing to not enable data or wifi, the time/timezone selection is
> still skipped.
> The only way to display the time/timezone selection screen is to remove the
> SIM card before beginning the FTE.
Seth, will you please file a new bug for this, and make it block this bug?
Flags: needinfo?(smiko)
Comment 20•10 years ago
|
||
(In reply to Michael Henretty [:mhenretty] from comment #19)
> Seth, will you please file a new bug for this, and make it block this bug?
Done: Bug 1063001
Flags: needinfo?(smiko)
Comment 21•10 years ago
|
||
Verified fixed. All depends on bugs are verified fixed, invalid, or duplicate.
Actual Result: date & time screen is skipped, when connected to a network.
Flame 2.2
Environmental Variables:
Device: Flame 2.2 Master (319mb)
BuildID: 20140905040204
Gaia: 5765c62163bcb7fde5ebfd211881117de31a7c46
Gecko: dddbe46f3ceb
Version: 35.0a1 (2.2 Master)
Firmware: V123
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0
Comment 22•10 years ago
|
||
Verified on Flame 2.1
Actual Result: date & time screen is skipped, when connected to a network.
Flame 2.1
Environmental Variables:
Device: Flame 2.1(319mb)
Build ID: 20140908000204
Gaia: a8e4d26555e5713ec6c72270cfd0cfabc096a0d3
Gecko: 746f24f9d21d
Version: 34.0a2 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
status-b2g-v2.2:
--- → verified
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
status-firefox34:
verified → ---
Flags: needinfo?(ktucker)
You need to log in
before you can comment on or make changes to this bug.
Description
•