-
Notifications
You must be signed in to change notification settings - Fork 81
DOCSP-45846-update-toggle-connections-compass #727
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-45846-update-toggle-connections-compass #727
Conversation
✅ Deploy Preview for docs-compass 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.
Thank you for these updates, @amalhotra-mdb! Overall, this LGTM from a copy standpoint.
I'm curious if the filter-connections.png screenshot could highlight the UI element. Maybe just a colored box around it, like in connection-sidebar.png. What do you think?
Thanks!
Joe
|
Great idea @jmd-mongo, I went ahead and added the red boxes to both screenshots! Thanks for your review, could you take a second look to make sure it looks ok? Thank you! |
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 these additional updates, @amalhotra-mdb! This one LGTM at this point.
Thanks,
Joe
DESCRIPTION
Update screenshots and add section to Compass "connection" page to demonstrate how to filter connections list for only active connections.
STAGING
https://deploy-preview-727--docs-compass.netlify.app/connect/connections/
JIRA
https://jira.mongodb.org/browse/DOCSP-45846
BUILD LOG
Self-Review Checklist
External Review Requirements
What's expected of an external reviewer?