Closed Bug 1061902 Opened 5 years ago Closed 5 years ago

[SettingsDB] Bump version number

Categories

(Core :: DOM: Device Interfaces, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
2.1 S4 (12sep)
Tracking Status
firefox33 --- wontfix
firefox34 --- fixed
firefox35 --- fixed
b2g-v2.1 --- fixed
b2g-v2.2 --- fixed

People

(Reporter: kgrandon, Assigned: kgrandon)

References

Details

(Keywords: verifyme, Whiteboard: [systemsfe])

Attachments

(1 file, 1 obsolete file)

Due to changed default settings in 2.1 we need to bump the version number in SettingsDB.jsm or users will not see these values.
This is a trivial change, and one I've made in the past so R=me.

On try in case it's needed: https://tbpl.mozilla.org/?tree=Try&rev=ada9d9d928b9
Attachment #8482964 - Flags: review+
Keywords: checkin-needed
Will do another patch...
Keywords: checkin-needed
Comment on attachment 8482964 [details] [diff] [review]
Patch - Upgrade SettingsDB version

Going to request a proper review.
Attachment #8482964 - Attachment is obsolete: true
Attachment #8482964 - Flags: review+
Gregor - Could you throw a review stamp on this? Thanks!
Attachment #8482973 - Flags: review?(anygregor)
Attachment #8482973 - Flags: review?(anygregor) → review+
Thanks for the review :) Adding checkin-needed again, there's a try run in comment 1 if needed.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/9196ec71a518
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Comment on attachment 8482973 [details] [diff] [review]
Patch - Upgrade SettingsDB version

Review of attachment 8482973 [details] [diff] [review]:
-----------------------------------------------------------------

This is a trivial change and is needed so default settings changes in gaia take effect after a 2.0 -> 2.1 upgrade. Please uplift to Aurora if possible, thanks!
Attachment #8482973 - Flags: approval-mozilla-aurora?
Keywords: verifyme
Attachment #8482973 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Unable to verify bug. This appears to be a backend issue with no STR.
QA Whiteboard: [QAnalyst-Triage?][QAnalyst-verify-]
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage?][QAnalyst-verify-] → [QAnalyst-Triage+][QAnalyst-verify-]
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage+][QAnalyst-verify-] → [QAnalyst-Triage+][QAnalyst-verify-] QAExclude
Flags: needinfo?(jmercado)
Flags: needinfo?(jmercado)
You need to log in before you can comment on or make changes to this bug.