Clarify nested fields #2585

guimachiavelli · 2023-10-17T16:38:20Z

Fixes #2525

github-actions · 2023-10-17T16:38:49Z

All broken links are now fixed, thank you!

Kerollmops

Thank you very much for the PR 😊

learn/inner_workings/datatypes.mdx

Co-authored-by: Clément Renault <[email protected]>

Kerollmops

Thank you very much! I do like that 😄

Kerollmops

I like that. Thank you 😄

guimachiavelli added 2 commits October 17, 2023 18:32

clarify distinction between nested fields and subdocument querying

5fdc025

💅

8c3eadf

shorten and clarify subdocument explanation

4f4c748

guimachiavelli marked this pull request as ready for review October 18, 2023 10:29

guimachiavelli requested a review from Kerollmops October 18, 2023 10:29

Kerollmops requested changes Oct 19, 2023

View reviewed changes

learn/inner_workings/datatypes.mdx Outdated Show resolved Hide resolved

learn/inner_workings/datatypes.mdx Show resolved Hide resolved

Update learn/inner_workings/datatypes.mdx

8945ff7

Co-authored-by: Clément Renault <[email protected]>

Kerollmops previously approved these changes Oct 19, 2023

View reviewed changes

address reviewer feedback

410fef6

guimachiavelli dismissed Kerollmops’s stale review via 410fef6 October 19, 2023 17:44

guimachiavelli requested a review from Kerollmops October 19, 2023 17:45

Kerollmops approved these changes Oct 23, 2023

View reviewed changes

guimachiavelli merged commit 118664a into main Oct 23, 2023
1 check passed

guimachiavelli deleted the clarify-nested-fields branch October 23, 2023 10:41

Provide feedback