Skip to content

DOCSP-18977 release notes for mongosh ticket#6054

Closed
ghost wants to merge 1 commit into
mongodb:masterfrom
mungitoperrito:DOCSP-18977-release-note-external-editor-v5.1
Closed

DOCSP-18977 release notes for mongosh ticket#6054
ghost wants to merge 1 commit into
mongodb:masterfrom
mungitoperrito:DOCSP-18977-release-note-external-editor-v5.1

Conversation

@ghost

@ghost ghost commented Oct 27, 2021

Copy link
Copy Markdown

No description provided.

@jeff-allen-mongo

Copy link
Copy Markdown
Collaborator

Hey @davemungo, was there a request to add this to the main server release notes?

We already have a note for this update in the mongosh release notes here. I'm confused why we aren't keeping this within the shell docs.

@ghost

ghost commented Oct 27, 2021

Copy link
Copy Markdown
Author

There wasn't a specific request, we could leave this one out.

The idea of adding it to the main docs is for visibility.We consider server and mongosh to be different entities but I don't know if users expect to have to check sub-components for updates.

@jeff-allen-mongo

Copy link
Copy Markdown
Collaborator

I see your point, however it seems odd to put this in the 5.1 release notes, since technically this feature is available on any version of MDB you're connecting with the shell to, provided your shell is 1.0 or greater.

I think this opens up confusion for users who think this feature would only work on server version 5.1 or greater.

@jeff-allen-mongo

Copy link
Copy Markdown
Collaborator

Would you be ok with closing this without merge, @davemungo ?

@ghost

ghost commented Oct 27, 2021

Copy link
Copy Markdown
Author

Closing - not needed

@ghost ghost closed this Oct 27, 2021
mongo-cr-bot pushed a commit that referenced this pull request Jan 29, 2024
…ions and move to an include (#5597) (#6054)

* Update required permissions and move to an include

* Simplify the permissions include

* fix typo and formatting

* add note about convention and move most to include
This pull request was closed.
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.

2 participants