-
Notifications
You must be signed in to change notification settings - Fork 59
DOCSP-48522-update-mongosh-history-log-description #394
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-48522-update-mongosh-history-log-description #394
Conversation
✅ Deploy Preview for docs-mongodb-shell ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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
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.
Thanks for picking this up @amalhotra-mdb! Just a couple small comments.
@@ -103,12 +103,16 @@ | |||
* - ``redactHistory`` | |||
- string | |||
- ``remove`` | |||
- Controls what information is recorded in the shell history. | |||
- Controls what information is recorded in the ``mongosh`` command history, | |||
meaning what can be accessed via up and down keys and Ctrl+R while inside the shell. |
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.
[suggestion] Instead of using a dependent clause here, I think we can just split this up into separate sentences to make it more concise/simple. Maybe something like:
Controls what information is recorded in the `mongosh` command history. You can access the `mongosh` command history via the up and down...
- Controls what information is recorded in the shell history. | ||
- Controls what information is recorded in the ``mongosh`` command history, | ||
meaning what can be accessed via up and down keys and Ctrl+R while inside the shell. | ||
|
||
Must be one of: |
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.
[suggestion] I know you didn't make this change, but it might be clearer if this introductory text doesn't end with a preposition
Must be one of: | |
Must be one of the following values: |
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 with one small nit!
Must be one of: | ||
- Controls what information is recorded in the ``mongosh`` command history. | ||
You can access the ``mongosh`` command history via the up and down keys | ||
and Ctrl+R while inside the shell. |
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.
[question/nit] Should we say "or" here since the Ctrl + R search mechanism in mongosh is separate from the up & down key navigation of the command history?
and Ctrl+R while inside the shell. | |
or Ctrl+R while inside the shell. |
DESCRIPTION
Update mongosh history log descriptions.
STAGING
JIRA
https://jira.mongodb.org/browse/DOCSP-48522
BUILD LOG
Self-Review Checklist
External Review Requirements
What's expected of an external reviewer?