-
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-48539-PreMigration-Analysis-API-endpoints #237
DOCSP-48539-PreMigration-Analysis-API-endpoints #237
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.
Thanks for making these updates @mmaville-mdb! 😄 Just a few comments to address before this should be ready to go with external review.
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.
Ah, sorry I preemptively approved the PR!
"database-type": { | ||
"type": "string" | ||
"description": "Database type or a custom database type. For example, `MYSQL`, `ORACLE`, `POSTGRESQL`, `SQL_SERVER`, `COCKROACHDB`, `DB2`, `SQLANYWHERE`, `SYBASE`, `SYBASEIQ`, or `YUGABYTE`.", |
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.
[praise] This seems much clearer! 🚀
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 ✅ Thanks for making these updates!
LGTM from Avinash via slack. |
DESCRIPTION
Adding Pre-Migration Analysis endpoints to the Relational Migrator API documentation.
This content is generated from mongodb-ets/migrator. I'll have the team edit the source and generate new output with corrections after initial copy review.
STAGING
https://deploy-preview-237--docs-relational-migrator.netlify.app/api-docs/rest-api-spec/
JIRA
https://jira.mongodb.org/browse/DOCSP-48539
Self-Review Checklist
External Review Requirements
What's expected of an external reviewer?