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

fix(serde_v8): serialize objects with numeric keys correctly #15946

Merged
merged 2 commits into from
Oct 2, 2022

Conversation

RaisinTen
Copy link
Contributor

Fixes: #11502
Signed-off-by: Darshan Sen raisinten@gmail.com

cc @bnoordhuis @AaronO

Fixes: denoland#11502
Signed-off-by: Darshan Sen <raisinten@gmail.com>
@RaisinTen RaisinTen changed the title fix(core): serialize objects with numeric keys correctly fix(serde_v8): serialize objects with numeric keys correctly Sep 25, 2022
@RaisinTen RaisinTen force-pushed the fix-core/parse-json-correctly branch from d5c881c to 10f0cac Compare September 25, 2022 08:06
@bartlomieju
Copy link
Member

Hey @RaisinTen sorry for slow response. I'm swamped with reviews needed for v1.26 release, so I'll come back to you PR next week.

Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you @RaisinTen nice work!

@bartlomieju bartlomieju merged commit bac3a12 into denoland:main Oct 2, 2022
@RaisinTen RaisinTen deleted the fix-core/parse-json-correctly branch October 3, 2022 04:18
hardfist pushed a commit to hardfist/deno that referenced this pull request Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error parsing javascript object to serde_json::Value
2 participants