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

The ID field should be first #7

Closed
baxiry opened this issue Nov 18, 2023 · 1 comment
Closed

The ID field should be first #7

baxiry opened this issue Nov 18, 2023 · 1 comment
Labels
fix later Not fatal
Milestone

Comments

@baxiry
Copy link
Owner

baxiry commented Nov 18, 2023

{
    "name": "adam",
    "age": 20,
    "_id": 4
}

I don't think _id should be first.
Also, having _id at the end improves performance slightly when parsing JSON.
However, we may make it optional and can be modified from the settings file.

@baxiry baxiry added the fix later Not fatal label Nov 18, 2023
@baxiry
Copy link
Owner Author

baxiry commented May 10, 2024

done

@baxiry baxiry closed this as completed May 10, 2024
@baxiry baxiry added this to the v0.6 milestone May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix later Not fatal
Projects
None yet
Development

No branches or pull requests

1 participant