-
Notifications
You must be signed in to change notification settings - Fork 21
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-46660-repetitive-paths #199
Conversation
✅ Deploy Preview for docs-relational-migrator 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.
Just one small comment!
@@ -150,9 +150,9 @@ Explore libraries and tools for MongoDB. | |||
Database Connections </database-connections/database-connections> | |||
|
|||
Projects </projects> | |||
Mapping Rules </mapping-rules/mapping-rules> | |||
Mapping Rules </mapping-rules/introduction> |
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.
[nit] just a heads up, I think we'll want to also add this page onto the snooty.toml file for toc landing pages.
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.
Fixed!
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.
LGTM! ✅
DESCRIPTION
Fixes the following repetitive paths:
https://www.mongodb.com/docs/relational-migrator/diagrams/diagrams/
https://www.mongodb.com/docs/relational-migrator/diagrams/navigate-diagrams/navigate-diagrams/
https://www.mongodb.com/docs/relational-migrator/connection-strings/connection-strings/
Also fixes build errors from #195
STAGING
https://deploy-preview-199--docs-relational-migrator.netlify.app/diagrams/
https://deploy-preview-199--docs-relational-migrator.netlify.app/connection-strings/
https://deploy-preview-199--docs-relational-migrator.netlify.app/diagrams/navigate-diagrams/
Result of mut-redirects config/redirects:
Redirect 301 /docs/relational-migrator/diagrams/diagrams/
https://www.mongodb.com/docs/relational-migrator/diagrams/
Redirect 301 /docs/relational-migrator/diagrams/navigate-diagrams/navigate-diagrams/ https://www.mongodb.com/docs/relational-migrator/diagrams/navigate-diagrams/
Redirect 301 /docs/relational-migrator/connection-strings/connection-strings/
https://www.mongodb.com/docs/relational-migrator/connection-strings/
JIRA
https://jira.mongodb.org/browse/DOCSP-46660
Self-Review Checklist
External Review Requirements
What's expected of an external reviewer?