Skip to content

DOCS-134 review #4

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
merged 1 commit into from
Feb 27, 2012
Merged

DOCS-134 review #4

merged 1 commit into from
Feb 27, 2012

Conversation

cwestin
Copy link

@cwestin cwestin commented Feb 25, 2012

Mostly I did small edits in place that clarify distinctions between values, objects/documents. In a couple of cases, there are larger changes I wasn't sure how you'd want to handle, so I marked them with TODO and made some comments.

While all the docs have been changed to use the shell's db..aggregate() shortcut, we should have something at the top of the reference that shows how to do this via db.runCommand(). Apparently the driver owners only just found out yesterday that they should be implementing this shortcut, and I'm worried not all of them will have it soon after 2.2 is released. Therefore, we should show the alternate command form so that users whose driver doesn't have the shortcut will know what to do. This has already come up, where users are now asking how to aggregate in drivers that have not been released with the shortcut.

tychoish added a commit that referenced this pull request Feb 27, 2012
@tychoish tychoish merged commit 124c882 into mongodb:master Feb 27, 2012
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 2, 2025
* DOCSP-44808-intellij-installation

* finish procedures

* *

* more feedback

* *

* add image of official plugin

* *
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 6, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 16, 2025
Makes approved changes from cloud-docs PR and resolves errors
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jun 26, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 2, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 15, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 15, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 17, 2025
Co-authored-by: Jordan Smith <45415425+jordan-smith721@users.noreply.github.com>
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 21, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 21, 2025
* DOCSP-46516: Setup

* more files

* test script

* edit

* JS feedback
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 21, 2025
* DOCSP-29036: Performance monitoring files

* most apm

* rest of apm methods
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 23, 2025
* DOCSP-12233: redirect pages in /upcoming navigation bar to /master
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 23, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 24, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 25, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 25, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 25, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 25, 2025
* DOCSP-38316: bson

* add meta tags

* CC suggestions
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 30, 2025
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 30, 2025
(cherry picked from commit b9a7270)
mongodb-server-docs-sync-bot bot pushed a commit that referenced this pull request Jul 30, 2025
* DOCSP-13827: add go quick-start

* wording changes

* nits

* Update source/includes/quick-start/main.go

Co-authored-by: Isabella Siu <sakurablossom@blueblueworld.com>

* remove pointless else

* comments

* addressing comments

* update query output

* Update source/quick-start.txt

Co-authored-by: Kevin Albertson <kevin.albertson@10gen.com>

* add another note for sample dataset

Co-authored-by: Isabella Siu <sakurablossom@blueblueworld.com>
Co-authored-by: Kevin Albertson <kevin.albertson@10gen.com>
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.

2 participants