Closed Bug 1737500 Opened 3 years ago Closed 3 years ago

mailchimp setup pages fails on loading

Categories

(Core :: JavaScript: Internationalization API, defect, P3)

Firefox 95
defect

Tracking

()

RESOLVED FIXED
101 Branch
Webcompat Priority P2
Tracking Status
firefox-esr78 --- unaffected
firefox-esr91 --- unaffected
firefox93 --- unaffected
firefox94 --- unaffected
firefox95 --- unaffected
firefox99 --- disabled
firefox100 --- disabled
firefox101 --- fixed

People

(Reporter: karlcow, Assigned: anba)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

  1. Go to https://mailchimp.com
  2. Create an account
  3. Wait for the confirmation email
  4. At a point the user is redirected to https://us5.admin.mailchimp.com/signup/setup/

The page fails with "an error occured"

When trying to diagnose a bit, we can see that the error is created by a RangeError.

and when running a mozregression, we get

19:27.49 INFO: Last good revision: d49d84396d57671c34657fa3bac0e385d6d07f70
19:27.49 INFO: First bad revision: 7f44a968af7fd604fc13de9a398dec4ce8ebebd2
19:27.49 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=d49d84396d57671c34657fa3bac0e385d6d07f70&tochange=7f44a968af7fd604fc13de9a398dec4ce8ebebd2

Which points to Bug 1648137

This was reported as a webcompat issue.
https://webcompat.com/issues/90899

It is happening in
https://us5.admin.mailchimp.com/release/1.1.12d10d170ea6367f…c4b36527f925bfd6e967169d/js-src/main.d2683f73469112c2ec6b.js

invalid value \"true\" for option useGrouping

at a point the code is reaching

    var a = {
      style: 'decimal',
      useGrouping: 'true'
    };

Thanks for the report! I've filed https://github.com/tc39/proposal-intl-numberformat-v3/issues/74 at the proposal repository.

Set release status flags based on info from the regressing bug 1648137

The feature which introduced this bug is Nightly-only, therefore I'm resetting the status fields for now.

Severity: -- → S2
Priority: -- → P3
Has Regression Range: --- → yes

(In reply to André Bargull [:anba] from comment #3)

The feature which introduced this bug is Nightly-only, therefore I'm resetting the status fields for now.

Is this still Nightly-only? Could you mark this bug as blocking the bug to enable this feature?

Flags: needinfo?(andrebargull)
Assignee: nobody → andrebargull
Status: NEW → ASSIGNED

We don't yet have a bug to enable this feature by default, but instead we can just fix the regression, now that the proposal has been updated.

Flags: needinfo?(andrebargull)
Pushed by andre.bargull@gmail.com: https://hg.mozilla.org/integration/autoland/rev/9e042f230960 Return fallback for unsupported values in GetStringOrBooleanOption. r=platform-i18n-reviewers,gregtatum
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: