Skip to content
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 37463 Updating connection window flow #647

Merged

Conversation

Copy link
Collaborator

@ajhuh-mdb ajhuh-mdb left a comment

Choose a reason for hiding this comment

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

Thanks @ltran-mdb2 for these updates! Just a few comments before this is ready to go.

@ltran-mdb2
Copy link
Collaborator Author

Thank you for your feedback @ajhuh-mdb ! This should be ready for another review.

@ajhuh-mdb
Copy link
Collaborator

LGTM ✅

@@ -25,6 +25,10 @@ Procedure
.. procedure::
:style: normal

.. step:: Open the :guilabel:`New Connection` window.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we referring to modal dialogs windows? Seems like that would be confusing given that window means something else already.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I changed all mentions to use the word "modal"

.. step:: (Optional). Name your connection.

Use the name field to enter a name for your connection. If you do
not specify a name, Compass uses the cluster's hostname as the the
Copy link
Collaborator

Choose a reason for hiding this comment

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

the the

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

.. step:: (Optional). Choose a color for your connection.

Use the color drop-down menu to select a label color for your
connection. When you activate a connection, the label color is the
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we going with "activate a connection"? Isn't it just connect and disconnect?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I used "activate" based on the "Active Connections" menu but changed to "connect"

@ltran-mdb2
Copy link
Collaborator Author

Thanks for your feedback @lerouxb ! Could you take another look when you get a chance?

@sarah-olson-mongodb sarah-olson-mongodb merged commit 3c8c981 into mongodb:multiple-connections Jul 22, 2024
1 check passed
@docs-builder-bot
Copy link

sarah-olson-mongodb pushed a commit that referenced this pull request Aug 27, 2024
* DOCSP 37462 Updating Connect to MonogDB Compass page for multiple connections (#617)

* DOCSP-37462 updating connect to MongoDB Compass page

* DOCSP-37462 updating connect to MongoDB Compass page

* DOCSP-37462 updating connect to MongoDB Compass page

* DOCSP-37462 removing extra space

* DOCSP 37458 - multiple connections on Compass Home (#616)

* DOCSP-37458 multiple connections home screen update

* DOCSP-37458 multiple connections home screen update

* DOCSP-37458 copy edits

* DOCSP-37458 typo

* DOCSP-37458 adding database title

* DOCSP-37458 adding database title

* DOCSP 37463 Updating connection window flow (#647)

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 general connection tab

* DOCSP-37463 copy edits

* DOCSP-37463 copy edits

* DOCSP-37463 tech edits

* DOCSP-37463 tech edits

* DOCSP-37461 Connections Sidebar Page (#646)

* DOCSP-37461 creating interact with your connections page

* DOCSP-37461 creating interact with your connections page

* DOCSP-37461 creating interact with your connections page

* DOCSP-37461 creating interact with your connections page

* DOCSP-37461 creating interact with your connections page

* DOCSP-37461 creating interact with your connections page

* DOCSP-37461 Ali comments

* DOCSP-37461 Ali comments

* DOCSP-37461 Ali comments

* Revert "DOCSP-37461 Ali comments"

This reverts commit b992c3a.

* Revert "DOCSP-37461 Ali comments"

This reverts commit 11bb3c6.

* DOCSP-37461 deleting extra page

* DOCSP-37461 copy edits

* DOCSP-37461 copy edits

* DOCSP-37461 typo

* DOCSP-37461 sidebar restructure

* DOCSP-37461 sidebar restructure

* DOCSP-37461 sidebar restructure

* DOCSP-37461 sidebar restructure

* DOCSP-37461 sidebar restructure

* DOCSP-37461 tech edits

* DOCSP-37461 tech edits

* DOCSP-37461 move shell option

* DOCSP 37460 Updating Compass Multiple Connections Screenshots (#658)

* DOCSP-37460 updating Compass screenshots

* DOCSP-37460 updating Compass screenshots

* DOCSP-37460 ian edits

* DOCSP-42696 update Favorite Connections and  Performance page (#661)

* DOCSP-42696 update Favorite Connections page

* DOCSP-42696 update Favorite Connections page

* DOCSP-42696 update Favorite Connections page

* DOCSP-42696 update Favorite Connections page

* DOCSP-42696 adding performance

* DOCSP-42696 adding performance

* DOCSP-42696 restructuring ToC

* DOCSP-42696 restructuring ToC

* DOCSP-38430 Julia comments

* DOCSP-38430 Julia comments

* DOCSP-38430 Julia comments
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.

5 participants