-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
DOCSP-43530: Id field in query results #3149
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm minor wording changes
docs/query-builder.txt
Outdated
code. When you run a find operation using the query builder, the {+odm-short+} | ||
automatically converts between ``id`` and ``_id``. In query results, | ||
the ``_id`` field is renamed to ``id``. This provides better |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
S: avoid using
code. When you run a find operation using the query builder, the {+odm-short+} | |
automatically converts between ``id`` and ``_id``. In query results, | |
the ``_id`` field is renamed to ``id``. This provides better | |
code. When you use the query builder to run a find operation, the {+odm-short+} | |
automatically converts between ``_id`` and``id`` field names. In query results, | |
the ``_id`` field is presented as ``id``. This provides better |
docs/query-builder.txt
Outdated
automatically converts between ``id`` and ``_id``. This provides better | ||
code. When you run a find operation using the query builder, the {+odm-short+} | ||
automatically converts between ``id`` and ``_id``. In query results, | ||
the ``_id`` field is renamed to ``id``. This provides better | ||
compatibility with Laravel, as the framework assumes that each record has a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
compatibility with Laravel, as the framework assumes that each record has a | |
consistency with Laravel, as the framework assumes that each record has a |
JIRA - https://jira.mongodb.org/browse/DOCSP-43530
Staging -
https://deploy-preview-138--docs-laravel.netlify.app/query-builder/#logical-or-example
https://deploy-preview-138--docs-laravel.netlify.app/upgrade/#version-5.x-breaking-changes
Checklist