Skip to content

Conversation

@ltran-mdb2
Copy link
Collaborator

@ltran-mdb2 ltran-mdb2 commented Oct 23, 2024

DESCRIPTION

Updates the validation screenshots and examples to use Atlas sample databases

STAGING

https://deploy-preview-685--docs-compass.netlify.app/validation/

JIRA

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

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 Oct 23, 2024

Deploy Preview for docs-compass ready!

Name Link
🔨 Latest commit d4c08f9
🔍 Latest deploy log https://app.netlify.com/sites/docs-compass/deploys/67191a92b538a90008b3ae22
😎 Deploy Preview https://deploy-preview-685--docs-compass.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

@jwilson-mdb jwilson-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 @ltran-mdb2, LGTM!

Copy link

@betsybutton betsybutton left a comment

Choose a reason for hiding this comment

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

LGTM, thank you Lauren! I also noticed that something else is outdated on this page. The limitation related to ADL should now reference ADF. Would it make sense to handle that change in a follow up ticket?

@ltran-mdb2
Copy link
Collaborator Author

I will open a follow up ticket! Thanks @betsybutton !

@ltran-mdb2 ltran-mdb2 merged commit 0258a61 into mongodb:master Oct 25, 2024
4 of 5 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