-
Notifications
You must be signed in to change notification settings - Fork 299
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-32303) includes info about connecting an application using driver to atlas #907
Conversation
Hi @carriecwk Can you include a link to the build? It also looks like a Vale check is failing, please take a look |
Would it be possible to incorporate the copy feedback from this review? Thanks! |
source/python.txt
Outdated
- `{+atlas+} | ||
<https://www.mongodb.com/docs/atlas?tck=docs_server>`__: The fully | ||
managed service for MongoDB deployments in the cloud | ||
- :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 |
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.
Hi @carriecwk, thanks for this change! Since this PR was opened I merged #910, which added an include we could use here. Could you please replace the bullets with the include? With that change this is good to go from me, thanks!
- `{+atlas+} | |
<https://www.mongodb.com/docs/atlas?tck=docs_server>`__: The fully | |
managed service for MongoDB deployments in the cloud | |
- :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 | |
.. include:: /includes/fact-environments.rst |
459ed37
to
ce7fc40
Compare
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, thank you!
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-32303
Staging - https://preview-mongodbcarriecwk.gatsbyjs.io/drivers/DOCSP-32303/python/
Build - https://workerpool-boxgs.mongodbstitch.com/pages/job.html?collName=queue&jobId=6525b179885de6b26b4f0d69
Self-Review Checklist