If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Missing 'else if' in conditional statement on hello/start.js

RESOLVED FIXED

Status

www.mozilla.org
Pages & Content
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: agibson, Assigned: agibson)

Tracking

Production
x86
All

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

3 years ago
There's a missing 'else if' on this line in the Hello FTE:

https://github.com/mozilla/bedrock/blob/master/media/js/firefox/hello/start.js#L239

Thankfully it doesn't result in anything unexpected happening, but we should fix this anyhow.
(Assignee)

Updated

3 years ago
Assignee: nobody → agibson
Status: NEW → ASSIGNED

Comment 1

3 years ago
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/d3a095f365668da03e34f9d43b310510dd91a852
[fix bug 1152783] Missing 'else if' in conditional statement on hello fte

https://github.com/mozilla/bedrock/commit/0e8530bcfa7e6e64e34a07b4738a61a09e11f117
Merge pull request #2900 from alexgibson/bug-1152783-fix-conditional-statement-hello-fte

[fix bug 1152783] Missing 'else if' in conditional statement on hello fte

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.