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-45089 clarifies mongodump/restore landing pages #175

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

jwilson-mdb
Copy link
Collaborator

@jwilson-mdb jwilson-mdb commented Nov 4, 2024

DESCRIPTION

Updates mongodump and mongorestore landing pages to clarify their behavior wrt version and FCV

STAGING

mongodump
mongorestore

JIRA

https://jira.mongodb.org/browse/DOCSP-45089

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Is this free of spelling errors?
  • Is this free of grammatical errors?
  • Is this free of staging / rendering issues?
  • Are all the links working?

External Review Requirements

What's expected of an external reviewer?

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for docs-commandline-tools ready!

Name Link
🔨 Latest commit b995874
🔍 Latest deploy log https://app.netlify.com/sites/docs-commandline-tools/deploys/672a574330c4e8000876d07b
😎 Deploy Preview https://deploy-preview-175--docs-commandline-tools.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@nvillahermosa-mdb nvillahermosa-mdb left a comment

Choose a reason for hiding this comment

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

Thanks Jonathan, LGTM!

Comment on lines 3 to 4
the source deployment. However, restoring files into a newer version
deployment is not the recommended way to upgrade your deployment. To learn
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 we should probably cut the last two sentences of this paragraph (starting with "however"). Since the support is now restricted to same major version or same FCV, I don't think we need to go into upgrading.

@jwilson-mdb jwilson-mdb requested a review from rohshar November 5, 2024 17:40
@jwilson-mdb jwilson-mdb merged commit 0258a71 into mongodb:master Nov 5, 2024
4 checks 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.

3 participants