Skip to content

Conversation

@nickldp
Copy link
Contributor

@nickldp nickldp commented Jul 14, 2023

Pull Request Info

PR Reviewing Guidelines

JIRA

Staging

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?

Copy link
Contributor

@rustagir rustagir left a comment

Choose a reason for hiding this comment

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

lgtm but address comments before merging!

Comment on lines 106 to 107
string by using the :ref:`Connect to MongoDB Atlas <connect-atlas-kotlin-driver>`
code example. Make sure to replace the connection string in the example.
Copy link
Contributor

Choose a reason for hiding this comment

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

The linked example has a different title than the one in the line

Suggested change
string by using the :ref:`Connect to MongoDB Atlas <connect-atlas-kotlin-driver>`
code example. Make sure to replace the connection string in the example.
string by using the :ref:`Atlas Connection Example <connect-atlas-kotlin-driver>`.
Make sure to replace the connection string in the example with yours.

Comment on lines 129 to 130
val mongoClient =
MongoClients.create("mongodb://<username>:<password>@<hostname>:<port>/?authSource=users");
Copy link
Contributor

Choose a reason for hiding this comment

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

I: make sure that your indentation doesn't affect your code examples. Also, I don't think this code example is correct, per the connection guide

Suggested change
val mongoClient =
MongoClients.create("mongodb://<username>:<password>@<hostname>:<port>/?authSource=users");
val mongoClient = MongoClient.create("mongodb://<username>:<password>@<hostname>:<port>/?authSource=users")

Error Sending Message
~~~~~~~~~~~~~~~~~~~~~

When you end a request through the driver and it is unable to send the command,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
When you end a request through the driver and it is unable to send the command,
When you send a request through the driver and it is unable to send the command,


===
FAQ
===
Copy link
Contributor

Choose a reason for hiding this comment

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

S: add a note to this page linking to the connection troubleshooting guide like the java one has: https://github.com/mongodb/docs-java/blob/6b87e5d49f837d6f1d9a16909d09df58828278a5/source/faq.txt#L18-L23

@nickldp nickldp merged commit ffc3935 into mongodb:master Jul 14, 2023
@docs-builder-bot
Copy link
Collaborator

nickldp added a commit to nickldp/docs-kotlin that referenced this pull request Jul 17, 2023
# Pull Request Info

[PR Reviewing
Guidelines](https://github.com/mongodb/docs-java/blob/master/REVIEWING.md)

## [JIRA ](https://jira.mongodb.org/browse/DOCSP-30463)

## [Staging
](https://docs-mongodbcom-staging.corp.mongodb.com/kotlin/docsworker-xlarge/DOCSP-30724-connection-troubleshooting-page/connection-troubleshooting/)

## Self-Review Checklist

- [x] Is this free of any warnings or errors in the RST?
- [x] Did you run a spell-check?
- [x] Did you run a grammar-check?
- [x] Are all the links working?

(cherry picked from commit ffc3935)
nickldp added a commit to nickldp/docs-kotlin that referenced this pull request Jul 17, 2023
# Pull Request Info

[PR Reviewing
Guidelines](https://github.com/mongodb/docs-java/blob/master/REVIEWING.md)

## [JIRA ](https://jira.mongodb.org/browse/DOCSP-30463)

## [Staging
](https://docs-mongodbcom-staging.corp.mongodb.com/kotlin/docsworker-xlarge/DOCSP-30724-connection-troubleshooting-page/connection-troubleshooting/)

## Self-Review Checklist

- [x] Is this free of any warnings or errors in the RST?
- [x] Did you run a spell-check?
- [x] Did you run a grammar-check?
- [x] Are all the links working?

(cherry picked from commit ffc3935)
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.

3 participants