Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Normative: Fix bug with RelativeTimeFormat and DateTimeFormat not correctly passing "numberingSystem" option to NumberFormat #919

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

ben-allen
Copy link
Contributor

Fix #819

This fixes a spec bug; as far as I can tell, all actually existing implementations do this the right way rather than the specified way. Will change title to start with "Normative:" if this category of fix requires TG1 consensus.

correctly passing "numberingSystem" option to NumberFormat
@ben-allen ben-allen added bug s: discuss Status: TG2 must discuss to move forward labels Aug 22, 2024
@ben-allen ben-allen changed the title Editorial: Fix bug with RelativeTimeFormat and DateTimeFormat not correctly passing "numberingSystem" option to NumberFormat Normative: Fix bug with RelativeTimeFormat and DateTimeFormat not correctly passing "numberingSystem" option to NumberFormat Aug 22, 2024
@ben-allen ben-allen added has consensus Has consensus from TC39-TG2 and removed s: discuss Status: TG2 must discuss to move forward labels Aug 29, 2024
@FrankYFTang
Copy link
Contributor

I believe this is presented to Oct TC39 meeting and still waiting for Mozilla to express their opinion.

@ben-allen
Copy link
Contributor Author

I believe they changed their conditional approval to approval at the start of the third day of plenary, but may have misunderstood. @codehag

@ben-allen
Copy link
Contributor Author

Received consensus at October 2024 TG1 meeting

@ben-allen ben-allen added needs tests has consensus (TG1) Has consensus from TC39-TG1 and removed needs consensus labels Oct 11, 2024
@ben-allen ben-allen merged commit 7508197 into tc39:main Oct 11, 2024
2 checks passed
@FrankYFTang
Copy link
Contributor

Should we also add test to test262 to test this PR got implement correctly?

@ben-allen
Copy link
Contributor Author

yes, I'll be writing them soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug has consensus (TG1) Has consensus from TC39-TG1 has consensus Has consensus from TC39-TG2 needs tests
Projects
Status: Previously Discussed
Development

Successfully merging this pull request may close these issues.

Numbering system option value not correctly passed to internal Intl.NumberFormat instances
3 participants