Skip to content
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

Merged
merged 5 commits into from
Oct 11, 2023

Conversation

carriecwk
Copy link
Contributor

@carriecwk carriecwk commented Sep 15, 2023

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

  • 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?

@sarahsimpers
Copy link
Contributor

Hi @carriecwk Can you include a link to the build? It also looks like a Vale check is failing, please take a look

@ccho-mongodb
Copy link
Contributor

Would it be possible to incorporate the copy feedback from this review? Thanks!

Comment on lines 38 to 44
- `{+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
Copy link
Contributor

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!

Suggested change
- `{+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

Copy link
Contributor

@sarahsimpers sarahsimpers left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@carriecwk carriecwk merged commit f04bd07 into mongodb:master Oct 11, 2023
1 check passed
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.

4 participants