Skip to content
This repository was archived by the owner on May 1, 2025. It is now read-only.

Conversation

@amalhotra-mdb
Copy link
Collaborator

@amalhotra-mdb amalhotra-mdb commented Nov 5, 2024

DESCRIPTION

As of 1.8.0, an invalid document filter in the start request will no longer crash mongosync; instead, the start request will fail and the user can correct the problem and re-issue the corrected start request without needing to restart mongosync. This information is now reflected in the document filtering page of the beta program in c2c sync.

STAGING

https://deploy-preview-464--docs-cluster-to-cluster-sync.netlify.app/reference/beta-program/document-filtering/#limitations

JIRA

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

BUILD LOG

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?

@netlify
Copy link

netlify bot commented Nov 5, 2024

Deploy Preview for docs-cluster-to-cluster-sync ready!

Name Link
🔨 Latest commit bce742d
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/672a3fa34a588a00080c97fd
😎 Deploy Preview https://deploy-preview-464--docs-cluster-to-cluster-sync.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.

LGTM!

Copy link

@tdq45gj tdq45gj left a comment

Choose a reason for hiding this comment

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

LGTM

@mdb-ashley mdb-ashley merged commit 28991ef into mongodb:master Nov 5, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants