(DOCSP-32271) Adds Atlas CTA to several Node.js pages #781

sarahsimpers · 2023-09-15T13:03:21Z

Pull Request Info

JIRA Tickets
https://jira.mongodb.org/browse/DOCSP-32271
https://jira.mongodb.org/browse/DOCSP-32273
https://jira.mongodb.org/browse/DOCSP-32274
https://jira.mongodb.org/browse/DOCSP-32275
https://jira.mongodb.org/browse/DOCSP-32294
https://jira.mongodb.org/browse/DOCSP-32306
https://jira.mongodb.org/browse/DOCSP-32305
https://jira.mongodb.org/browse/DOCSP-32307

Build
https://workerpool-boxgs.mongodbstitch.com/pages/job.html?collName=queue&jobId=65046d9724fcc731b473939c

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?

mongoKart

I = Issue
S = Suggestion
Q = Question

mongoKart · 2023-09-15T14:06:31Z

source/includes/fact-environments.rst

+- :ref:`MongoDB Enterprise <install-mdb-enterprise>`: The
+  subscription-based, self-managed version of MongoDB
+- :ref:`MongoDB Community <install-mdb-community-edition>`: The
+  source available, free-to-use, and self-managed version of MongoDB


I: "source available" is a compound modifier and should be hyphenated

Suggested change

source available, free-to-use, and self-managed version of MongoDB

source-available, free-to-use, and self-managed version of MongoDB

mongoKart · 2023-09-15T14:09:25Z

source/index.txt

+You can connect using the Node.js driver for
+deployments hosted in the following environments:


I: I don't love that the Style Guide makes this an absolute, but it prohibits standalone "using". I think something like this is clearer, too:

Suggested change

You can connect using the Node.js driver for

deployments hosted in the following environments:

You can use the Node.js driver to connect to

deployments hosted in the following environments:

mongoKart · 2023-09-15T14:10:44Z

source/index.txt

@@ -32,6 +32,11 @@ installing the {+driver-short+}, see
 :ref:`Download and Install <node-quick-start-download-and-install>` in the
 Quick Start guide.

+You can connect using the Node.js driver for


S: We have a source constant for "Node.js driver'

Suggested change

You can connect using the Node.js driver for

You can connect using the {+driver-short+} for

mongoKart · 2023-09-15T14:13:04Z

source/quick-reference.txt

+-------------
+
+.. |page-topic| replace:: use the following driver syntax
+.. |link-topic-ing| replace:: using common CRUD operations in the Atlas UI


S:

Suggested change

.. |link-topic-ing| replace:: using common CRUD operations in the Atlas UI

.. |link-topic-ing| replace:: performing common CRUD operations in the Atlas UI

mongoKart · 2023-09-15T14:15:02Z

source/includes/fact-atlas-compatible.rst

@@ -0,0 +1,4 @@
+You can connect using the Node.js driver and |page-topic| for


I: To avoid "using" and also make this a little clearer, I'd go with something like this:

Suggested change

You can connect using the Node.js driver and |page-topic| for

You can use the {+driver-short+} to connect and |page-topic| for

mongoKart · 2023-09-15T14:17:36Z

source/fundamentals/crud.txt

+-------------
+
+.. |page-topic| replace:: use CRUD operations
+.. |link-topic-ing| replace:: using CRUD operations in the Atlas UI


Suggested change

.. |link-topic-ing| replace:: using CRUD operations in the Atlas UI

.. |link-topic-ing| replace:: performing CRUD operations in the Atlas UI

mongoKart · 2023-09-15T14:18:40Z

source/fundamentals/connection.txt

+Compatibility
+-------------
+
+.. |page-topic| replace:: use the Node.js driver


S: snooty constant for "Node.js driver"

Suggested change

.. |page-topic| replace:: use the Node.js driver

.. |page-topic| replace:: use the {+driver-short+}

mongoKart · 2023-09-15T14:20:03Z

source/fundamentals/connection.txt

+-------------
+
+.. |page-topic| replace:: use the Node.js driver
+.. |link-topic-ing| replace:: connecting using drivers


I: standalone "using"

S:

Suggested change

.. |link-topic-ing| replace:: connecting using drivers

.. |link-topic-ing| replace:: using drivers to connect

(still standalone, but unambiguous here imo)

mongoKart · 2023-09-15T14:30:43Z

source/quick-reference.txt

+Compatibility
+-------------
+
+.. |page-topic| replace:: use the following driver syntax


I: This page-topic makes the final sentence a little awkward, especially if you accept my suggestion in an earlier comment:

"You can use the Node.js driver to connect and [use the following driver syntax] for deployments hosted in the following environments:"

S:

Suggested change

.. |page-topic| replace:: use the following driver syntax

.. |page-topic| replace:: execute commands

^^^ This is how the page intro describes the content of the page

mongoKart · 2023-09-15T14:33:58Z

source/fundamentals/crud/read-operations/retrieve.txt

+.. |page-topic| replace:: use read operations
+.. |link-topic-ing| replace:: using read operations in the Atlas UI
+
+.. |atlas-url| replace:: :atlas:`Create, View, Update, and Delete Documents </atlas-ui/documents/#view--filter--and-sort-documents>`


Q: Do you think it makes sense to use a different link title since this goes to the middle of the page?

mongoKart

LGTM!

sarahsimpers force-pushed the DOCSP-32271 branch 2 times, most recently from dfc878a to 9240101 Compare September 15, 2023 13:06

(DOCSP-32271) Adds Atlas CTA to several Node.js pages

98726b9

sarahsimpers force-pushed the DOCSP-32271 branch from 9240101 to 98726b9 Compare September 15, 2023 13:09

MatteoVH approved these changes Sep 15, 2023

View reviewed changes

Removes via from page name

9e6cb16

mongoKart requested changes Sep 15, 2023

View reviewed changes

Includes changes from copy review

41cef14

sarahsimpers requested a review from mongoKart September 15, 2023 14:43

mongoKart approved these changes Sep 15, 2023

View reviewed changes

mongoKart merged commit 6b2f49f into mongodb:master Sep 15, 2023

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

765e10c

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v6.0 #782

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

ac4575a

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.8 #783

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

7eafb67

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.7 #784

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

ea3610e

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.6 #785

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

41db25e

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.5 #786

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

78074f6

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.4 #787

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

ea521e5

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.3 #788

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

1d448bd

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.2 #789

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

8bfe60c

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v5.1 #790

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

d29593b

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

d00fdc2

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v4.12 #797

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

f03b523

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v4.11 #798

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

5a559ef

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v4.10 #799

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

90e174a

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v4.9 #800

Merged

sarahsimpers added a commit to sarahsimpers/docs-node that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

bed2238

sarahsimpers mentioned this pull request Sep 15, 2023

Backports DOCSP-32271 to v4.8 #801

Merged

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#801)

c2cf2f2

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#800)

4b2b382

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#799)

5810648

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#798)

eb582f5

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#797)

9c13a67

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#796)

3e76c06

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#795)

3a17d88

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#794)

87c9da0

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#793)

2dbe05b

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#792)

a4f0eb7

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#791)

3c5ec74

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#789)

b4ae4ca

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#788)

a5a44d0

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#787)

c9a35b1

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#786)

830749e

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#784)

77214db

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#783)

a0f680a

mongoKart pushed a commit that referenced this pull request Sep 15, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (#781) (#782)

504ee6f

ccho-mongodb mentioned this pull request Sep 18, 2023

(DOCSP-32303) includes info about connecting an application using driver to atlas mongodb/docs-ecosystem#907

Merged

4 tasks

mongoKart pushed a commit to mongoKart/docs-node that referenced this pull request Nov 3, 2023

(DOCSP-32271) Adds Atlas CTA to several Node.js pages (mongodb#781)

c14ce95

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

(DOCSP-32271) Adds Atlas CTA to several Node.js pages #781

(DOCSP-32271) Adds Atlas CTA to several Node.js pages #781

Uh oh!

sarahsimpers commented Sep 15, 2023 •

edited

Loading

Uh oh!

mongoKart left a comment

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart Sep 15, 2023

Uh oh!

mongoKart left a comment

Uh oh!

Uh oh!

	source available, free-to-use, and self-managed version of MongoDB
	source-available, free-to-use, and self-managed version of MongoDB

		You can connect using the Node.js driver for
		deployments hosted in the following environments:

	You can connect using the Node.js driver for
	You can connect using the {+driver-short+} for

	.. \|link-topic-ing\| replace:: using common CRUD operations in the Atlas UI
	.. \|link-topic-ing\| replace:: performing common CRUD operations in the Atlas UI

		@@ -0,0 +1,4 @@
		You can connect using the Node.js driver and \|page-topic\| for

	You can connect using the Node.js driver and \|page-topic\| for
	You can use the {+driver-short+} to connect and \|page-topic\| for

	.. \|link-topic-ing\| replace:: using CRUD operations in the Atlas UI
	.. \|link-topic-ing\| replace:: performing CRUD operations in the Atlas UI

	.. \|page-topic\| replace:: use the Node.js driver
	.. \|page-topic\| replace:: use the {+driver-short+}

	.. \|link-topic-ing\| replace:: connecting using drivers
	.. \|link-topic-ing\| replace:: using drivers to connect

	.. \|page-topic\| replace:: use the following driver syntax
	.. \|page-topic\| replace:: execute commands

(DOCSP-32271) Adds Atlas CTA to several Node.js pages #781

(DOCSP-32271) Adds Atlas CTA to several Node.js pages #781

Uh oh!

Conversation

sarahsimpers commented Sep 15, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request Info

Self-Review Checklist

Uh oh!

mongoKart left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mongoKart left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

sarahsimpers commented Sep 15, 2023 •

edited

Loading