Closed
Bug 1953199
Opened 1 year ago
Closed 1 year ago
Reword JSMSG_TEMPORAL_PLAIN_DATE_INVALID 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: ltaylor, Mentored)
References
(Blocks 1 open bug)
Details
(Keywords: good-first-bug)
Attachments
(1 file)
The JSMSG_TEMPORAL_PLAIN_DATE_INVALID is used to report invalid day, month or year fields to the user.
The current wording is "year-month-day must be valid iso dates", I think it would be more helpful to say "year-month-day must be valid ISO date values".
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_PLAIN_DATE_INVALID 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 → luke
Status: NEW → ASSIGNED
Pushed by dminor@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/288832407495
Reword JSMSG_TEMPORAL_PLAIN_DATE_INVALID error message; r=dminor
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
•