Closed
Bug 1951737
Opened 1 year ago
Closed 1 year ago
Reword JSMSG_TEMPORAL_DURATION_INVALID_NORMALIZED_TIME error message.
Categories
(Core :: JavaScript: Standard Library, enhancement, P3)
Core
JavaScript: Standard Library
Tracking
()
RESOLVED
FIXED
138 Branch
| Tracking | Status | |
|---|---|---|
| firefox138 | --- | fixed |
People
(Reporter: dminor, Assigned: tannal2409, Mentored)
References
(Blocks 1 open bug)
Details
(Keywords: good-first-bug)
Attachments
(1 file)
The JSMSG_TEMPORAL_DURATION_INVALID_NORMALIZED_TIME error is used to report a too large time to the developer. The current wording is "time duration values must be lower than 253 seconds", I think it would be better to say "time duration values must be smaller than 253 seconds".
This is a good first bug, the main complexity will be getting the SpiderMonkey / Firefox build working.
- Make sure you can build SpiderMonkey. You can find instructions on building here: https://firefox-source-docs.mozilla.org/js/build.html.
- Change the error message, and then make sure the build is still working and you can run the tests. You might need to make some changes to the tests if they were expecting the old error message.
- Now you can commit your changes. Your commit message should look like:
Bug X - Reword JSMSG_TEMPORAL_DURATION_INVALID_NORMALIZED_TIME error message; r=dminor!, where X is the number for this bug. - Once you have your changes committed, request review. There are instructions on using phabricator and on getting reviews.
Updated•1 year ago
|
Severity: -- → N/A
Priority: -- → P3
| Assignee | ||
Comment 1•1 year ago
|
||
Updated•1 year ago
|
Assignee: nobody → tannal2409
Status: NEW → ASSIGNED
Pushed by dminor@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0f2719e49b12
Reword JSMSG_TEMPORAL_DURATION_INVALID_NORMALIZED_TIME error message; r=dminor
Comment 3•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox138:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•