Skip to content

DOCSP-47796: Database connections#230

Merged
mayaraman19 merged 2 commits intomongodb:docs-upliftfrom
mayaraman19:DOCSP-47796-db-connections
Mar 24, 2025
Merged

DOCSP-47796: Database connections#230
mayaraman19 merged 2 commits intomongodb:docs-upliftfrom
mayaraman19:DOCSP-47796-db-connections

Conversation

@mayaraman19
Copy link
Collaborator

@mayaraman19 mayaraman19 commented Mar 18, 2025

DESCRIPTION

Adds four places in UI to create a database connection.

STAGING

https://deploy-preview-230--docs-relational-migrator.netlify.app/database-connections/manage-database-connections/#create-a-connection

JIRA

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

BUILD LOG

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 Mar 18, 2025

Deploy Preview for docs-relational-migrator ready!

Name Link
🔨 Latest commit 4221995
🔍 Latest deploy log https://app.netlify.com/sites/docs-relational-migrator/deploys/67e1a2841f087400085d9fe3
😎 Deploy Preview https://deploy-preview-230--docs-relational-migrator.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 30 (🔴 down 42 from production)
Accessibility: 85 (🔴 down 13 from production)
Best Practices: 75 (no change from production)
SEO: 75 (🔴 down 8 from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@gmiller-mdb gmiller-mdb left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

One requested update in the review session. LGTM'd, so no need for additional review after making the requested change.

-------------------

You can create a connection by using one of the following places in the
Relational Migrator UI.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Feedback from review session: Can we add a link or a sentence w/a link to the UI Overview page?

This would help newer users who aren't as familiar with RM identify the UI elements we list below.

@mayaraman19 mayaraman19 merged commit f74c411 into mongodb:docs-uplift Mar 24, 2025
5 checks passed
@mayaraman19 mayaraman19 deleted the DOCSP-47796-db-connections branch March 24, 2025 19:01
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