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

DOCSP-46301-clean-verify-data #617

Merged

Conversation

gmiller-mdb
Copy link
Collaborator

@gmiller-mdb gmiller-mdb commented Feb 13, 2025

DESCRIPTION

Created new page for data verification.

STAGING

https://deploy-preview-617--docs-cluster-to-cluster-sync.netlify.app/reference/verification/

https://deploy-preview-617--docs-cluster-to-cluster-sync.netlify.app/reference/verification/custom-manual/#std-label-c2c-custom-manual-verification

JIRA

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

SELF-REVIEW CHECKLIST

  • Does each file have 3-5 taxonomy facet tags?
    See the taxonomy tagging instructions and this example PR
  • 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 Feb 13, 2025

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

Name Link
🔨 Latest commit 9ba6958
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/67be318bd5a03b00089081fc
😎 Deploy Preview https://deploy-preview-617--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.

@nvillahermosa-mdb nvillahermosa-mdb self-requested a review February 14, 2025 21:28
Copy link
Collaborator

@rohshar rohshar left a comment

Choose a reason for hiding this comment

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

in the table, I'd put Migration Verifier above Custom or Manual Verification.

I think you can also have some of the content from custom-manual.txt in the table, if that helps with SEO. Maybe just say you can verify documents, indexes, metdata, shard keys and include links to the sections in the page

Copy link
Collaborator

@rohshar rohshar left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

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.

@gmiller-mdb Thanks for following through on this! The new page needs a title and short description, and aside from that I left a few stylistic edits. Happy to chat in Slack/Zoom if you'd like.

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 @gmiller-mdb , LGTM!

Copy link
Collaborator

@rohshar rohshar left a comment

Choose a reason for hiding this comment

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

still LGTM

@gmiller-mdb gmiller-mdb merged commit f39c8db into mongodb:master Mar 3, 2025
4 checks passed
gmiller-mdb added a commit to gmiller-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 3, 2025
* DOCSP-46301-clean-verify-data

* move code

* fix indent errors

* changing table

* fix links

* feedback

* build errors
gmiller-mdb added a commit to gmiller-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 3, 2025
* DOCSP-46301-clean-verify-data

* move code

* fix indent errors

* changing table

* fix links

* feedback

* build errors
gmiller-mdb added a commit that referenced this pull request Mar 3, 2025
* DOCSP-46301-clean-verify-data

* move code

* fix indent errors

* changing table

* fix links

* feedback

* build errors
gmiller-mdb added a commit that referenced this pull request Mar 3, 2025
* DOCSP-46301-clean-verify-data

* move code

* fix indent errors

* changing table

* fix links

* feedback

* build errors
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.

3 participants