Skip to content

(DOCS-10422): Added information on file extension visibility. #3345

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

Merged
merged 2 commits into from
Jun 19, 2018

Conversation

atsansone
Copy link
Contributor

@rkumar-mongo : This is a simple note that tells users to make sure to turn off "Hide Known File Extensions."

Copy link
Contributor

@ravindk89 ravindk89 left a comment

Choose a reason for hiding this comment

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

Two small requested changes.

@@ -44,6 +44,16 @@ enter the following command in either the :guilabel:`Command Prompt` or
OS Version: 10.0.17134 N/A Build 17134
System Type: x64-based PC

Make certain that you show set Windows Explorer to show file name
Copy link
Contributor

Choose a reason for hiding this comment

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

"show set" ?

@@ -44,6 +44,16 @@ enter the following command in either the :guilabel:`Command Prompt` or
OS Version: 10.0.17134 N/A Build 17134
System Type: x64-based PC

Make certain that you show set Windows Explorer to show file name
extensions for all file types. This can prevent issues where the file
extension that is shown is not the file extension that Windows
Copy link
Contributor

Choose a reason for hiding this comment

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

where the file extension shown

@@ -44,6 +44,16 @@ enter the following command in either the :guilabel:`Command Prompt` or
OS Version: 10.0.17134 N/A Build 17134
System Type: x64-based PC

Make certain that you show set Windows Explorer to show file name
extensions for all file types. This can prevent issues where the file
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is closer to:

"This can prevent issues where the file type displayed to the user differs from the actual file type."

@atsansone
Copy link
Contributor Author

@rkumar-mongo : Changes made. Appreciate your second review.

Copy link
Contributor

@ravindk89 ravindk89 left a comment

Choose a reason for hiding this comment

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

LGTM

@atsansone atsansone assigned kay-kim and unassigned ravindk89 Jun 18, 2018
@atsansone atsansone added RFM and removed Copy Review labels Jun 18, 2018
@atsansone
Copy link
Contributor Author

@kay-kim : RFM for v3.6 and master.

@kay-kim kay-kim merged commit ac2e9a2 into mongodb:master Jun 19, 2018
mongo-cr-bot pushed a commit that referenced this pull request Jun 14, 2023
* DOCS-16154 Fixes typo in Server Parameters

* Fixes build issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants