Skip to content

DOCSP-29510: explain how driver handles undefined vals #674

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

Merged

Conversation

rustagir
Copy link
Collaborator

@rustagir rustagir commented May 3, 2023

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-29510
Staging:
Landing page for new fundamentals section

Undefined values page

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Did you run a spell-check?
  • Did you run a grammar-check?
  • Are all the links working?

@@ -205,3 +205,5 @@ inserts a new document with the specified fields and values.
You cannot modify the ``_id`` field of a document nor change a field to
a value that violates a unique index constraint. See the MongoDB Server manual
for more information on :manual:`unique indexes </core/index-unique/>`.

.. include:: /includes/undefined-values-note.rst
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm undecided on whether this was the right approach, as opposed to making different content for these pages based on the type of write op that they explain. I also couldn't find the right place to put this material, as it didn't seem appropriate to make a whole subsection for it on each page.

Copy link
Collaborator

Choose a reason for hiding this comment

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

We have the UTF-8 Validation docs as a top level page but this and that UTF page are both BSON controls. Maybe there's a merging or nesting that would be sensical given that.

I foresee a near future where we may be adding more coverage of the BSON controls that the driver offers, we just have to make the options more easily understood first.

Just a suggestion, of course!

@rustagir rustagir marked this pull request as draft May 3, 2023 20:41
@rustagir rustagir marked this pull request as ready for review May 4, 2023 20:07
@@ -17,3 +17,4 @@ raw: ${prefix}/stable -> ${base}/current/
[*-v5.0]: ${prefix}/${version}/quick-start/download-and-install/ -> ${base}/${version}/quick-start/
[*-v5.0]: ${prefix}/${version}/quick-start/next-steps/ -> ${base}/${version}/quick-start/

[v4.3-master]: ${prefix}/${version}/fundamentals/utf8-validation -> ${base}/${version}/fundamentals/bson/utf8-validation/
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@jordan-smith721 jordan-smith721 left a comment

Choose a reason for hiding this comment

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

Nice work on this! I've got a few suggestions that I think could make the page a bit easier to digest with how technical it is.

@rustagir rustagir requested a review from jordan-smith721 May 5, 2023 17:24
Copy link
Collaborator

@jordan-smith721 jordan-smith721 left a comment

Choose a reason for hiding this comment

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

Nice work with the second pass! LGTM with one small suggestion

Comment on lines 30 to 33
The following example inserts two documents. One insert operation has
the ``ignoreUndefined`` setting set to ``true``. The driver does not
serialize the ``salesTax`` field during the first insert operation
because we specified the ``ignoreUndefined`` setting:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggestion: I think these sentences can be combined:

Suggested change
The following example inserts two documents. One insert operation has
the ``ignoreUndefined`` setting set to ``true``. The driver does not
serialize the ``salesTax`` field during the first insert operation
because we specified the ``ignoreUndefined`` setting:
The following example inserts two documents. The first insert operation has
the ``ignoreUndefined`` setting set to ``true``, so it does not
serialize the ``salesTax`` field.

@rustagir rustagir merged commit 52daa59 into mongodb:master May 5, 2023
rustagir added a commit that referenced this pull request May 5, 2023
* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)
rustagir added a commit that referenced this pull request May 5, 2023
* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)
rustagir added a commit that referenced this pull request May 5, 2023
* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)
rustagir added a commit that referenced this pull request May 5, 2023
* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)
rustagir added a commit to rustagir/docs-node that referenced this pull request May 5, 2023
* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)
rustagir added a commit that referenced this pull request May 5, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert
rustagir added a commit to rustagir/docs-node that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (mongodb#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
rustagir added a commit to rustagir/docs-node that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (mongodb#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
(cherry picked from commit d3b2847)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
(cherry picked from commit d3b2847)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
(cherry picked from commit d3b2847)
rustagir added a commit that referenced this pull request May 8, 2023
* DOCSP-29510: explain how driver handles undefined vals (#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
(cherry picked from commit d3b2847)
(cherry picked from commit 5a50abe)
jordan-smith721 pushed a commit to jordan-smith721/docs-node that referenced this pull request Jul 24, 2023
* DOCSP-29510: explain how driver handles undefined vals (mongodb#674)

* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion

(cherry picked from commit 52daa59)

* redirect file revert

(cherry picked from commit 157f483)
(cherry picked from commit f7c1609)
(cherry picked from commit 82a8fff)
(cherry picked from commit d2b072b)
(cherry picked from commit d3b2847)
mongoKart pushed a commit to mongoKart/docs-node that referenced this pull request Nov 3, 2023
* DOCSP-29510: explain how driver handles undefined vals

* make new subfolder for bson

* tree fixes and redirect-- snooty stuff

* fix version in redirect

* fix highlight

* JS PR fixes 1

* JS suggestion
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.

3 participants