818348 - [FTU] [Settings] Share info filled during FTU (i.e. "Improve Firefox OS" checkbox)

Status: RESOLVED FIXED

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

Description

[Daniel Holbert [:dholbert]]

I just re-flashed with the unagi_beta_2012-12-03.zip dogfood build, and during the first run experience, I ticked the checkbox for "Submit performance data" etc. in the second-to-last screen.

Then, after completing that process, I went over to settings and checked the "Improve Firefox OS" section. I expect that the "Submit Performance Data" checkbox should be checked, but it's not.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: screenshot of box that I checked during first-run experience

Here's a screenshot of the first-run experience page where I ticked the box.

Comment 2

[Daniel Holbert [:dholbert]]

Attachment: screenshot of "Improve Firefox OS" settings page w/ box un-ticked

...and here's a screenshot of the "Improve Firefox OS" page w/ "submit performance data" un-ticked (as it appeared when I first visited it, after completing first run experience)

Comment 3

[David Scravaglieri [:scravag]]

Borja renominate if needed

Comment 4

[Borja Salguero [:borjasalguero]]

I've renamed the bug because the problem is that currently we have no mechanism to share the info filled during FTU to 'Settings' app. Probably we could use mozSettings, but I would like to know the opinion of Kaze.

Comment 5

[Fabien Cazenave [:kaze]]

Yup, a mozSetting would make sense.

Comment 6

[Borja Salguero [:borjasalguero]]

Vivien would not be so happy adding a lot of stuff in mozSetting, I would like to be sure that we agree on it.

Comment 7

[Maria Angeles Oteo (:oteo)]

(In reply to dscravaglieri from comment #3)
> Borja renominate if needed

After testing it in my device, it should be BB+ as we need to have updated the info modified in FTU in the settings app.

Comment 8

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Borja Salguero [:borjasalguero] (limited access to email until 10th December) from comment #6)
> Vivien would not be so happy adding a lot of stuff in mozSetting, I would
> like to be sure that we agree on it.

I thought this one was already a mozSettings. I am 100% fine with this one beeing a mozSetting. (The one I'm worried about beeing the email of the last section).

Comment 9

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Gaia Triage : + , seems like broken functionality.

Comment 10

[Dietrich Ayala (:dietrich)]

Been over a week - what needs to happen here to fix this?

Comment 11

[Fernando Campo (:fcampo)]

Stealing :) as my other blockers are on discussion and I have some time right now

Comment 12

[Fernando Campo (:fcampo)]

Attachment: link to https://github.com/mozilla-b2g/gaia/pull/7026

Comment 13

[Fernando Campo (:fcampo)]

https://github.com/mozilla-b2g/gaia/commit/e38fe1e99841b0515b218851211234141bfc4f35