-
Notifications
You must be signed in to change notification settings - Fork 11
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
[C] Duplicate Titles - Connect to MongoDB #93
Conversation
✅ Deploy Preview for docs-c ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good! Left comments about capitalizing the titles showing up in the page TOC. Also not sure if the Connect to MongoDB section is staging correctly (it isn't showing up at all on my end)
source/get-started.txt
Outdated
.. _c-get-started-connect-to-mongodb: | ||
|
||
Connect to MongoDB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I: This section isn't showing up in staging for some reason :/ Not sure if it's just on my end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm, this is showing up for me (also, I renamed it, Run a Sample Query)
https://deploy-preview-93--docs-c.netlify.app/get-started/#run-a-sample-query
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok I see it now! Netlify strikes again :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks LGTM!
(cherry picked from commit 5717c5c)
(cherry picked from commit 5717c5c)
(cherry picked from commit 5717c5c) Co-authored-by: Rachel Mackintosh <[email protected]>
(cherry picked from commit 5717c5c) Co-authored-by: Rachel Mackintosh <[email protected]>
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-48229
Merging the Get Started section into one page, per the new TOC guidelines
Ignoring language-related vale complaints because this is all c+p.
Staging - https://deploy-preview-93--docs-c.netlify.app/get-started/
Self-Review Checklist