Skip to content

DOCSP-42971-vscode-nested-components #93

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.

Sign up for GitHub

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

Merged
merged 5 commits into from
Sep 16, 2024
Merged

DOCSP-42971-vscode-nested-components #93

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d88fb3a
DOCSP-42971-vscode-nested-components
gmiller-mdb Sep 13, 2024
20574d8
took out extranous line
gmiller-mdb Sep 13, 2024
43dc962
build errors
gmiller-mdb Sep 16, 2024
6fa8932
more build errors
gmiller-mdb Sep 16, 2024
0a2ccd2
be
gmiller-mdb Sep 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions source/commands.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,8 @@ To learn more about connecting to a deployment, see :ref:`vsce-connect`.
- Select and remove a connection from your list of connections
in |vsce|.

.. note::

Removing your current active connection also closes that
connection.
Removing your current active connection also closes that
connection.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I can't comment on the applicable lines, but on this file and on settings.txt, build errors are being caused by the |vsce-short| substitution. Can you check to make sure this variable exists in snooty.toml? If not, replace the variable with |vsce| !

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ajhuh-mdb Tried to fix the build errors seems like the new command is 'vscode-short'. Hopefully that works!

Shell Commands
--------------
Expand Down
2 changes: 0 additions & 2 deletions source/includes/admonitions/warn-configure-path.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
.. important::

To launch your selected MongoDB shell, you must:
Copy link
Collaborator

Choose a reason for hiding this comment

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

[nit] The indent(s) here are causing a build error!


Expand All @@ -10,7 +9,6 @@
If you install a new version of your MongoDB shell, you will need to
update your ``PATH`` variable to reference the new version.

.. seealso::

- To install the legacy :binary:`mongo <mongo>`
shell, see :manual:`The mongo Shell </mongo>`.
Expand Down
21 changes: 10 additions & 11 deletions source/includes/steps-starting-vsce-connect-with-advanced-connection-settings.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,17 +157,17 @@ content: |
specify the :guilabel:`Hostname`. You do not need to
specify a port.

.. note::


You can only use SRV connection strings to connect to
deployments running MongoDB 3.6 and later.
You can only use SRV connection strings to connect to
Copy link
Contributor

Choose a reason for hiding this comment

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

[suggestion]

I think we can consider removing this line entirely since this covers all supported versions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jmd-mongo this look good?

deployments running MongoDB 3.6 and later.

SRV connection strings have a prefix of
``mongodb+srv:``. If you are using an SRV connection
string, you do not need to include ``mongodb+srv`` in
your :guilabel:`Hostname`.

.. example::
For example:

If the following is your SRV connection string for
Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe the indents here are also causing build errors!

your MongoDB deployment:
Expand Down Expand Up @@ -258,14 +258,13 @@ content: |
provide the :guilabel:`Username` to authenticate
the user.

.. note::

If you are using
:atlas:`Atlas-managed certificates </security-add-mongodb-users/#select-an-authentication-method>`
, your username must be prefaced by "CN="
per `RFC 2253 <https://tools.ietf.org/html/rfc2253>`__.
For example, the username "X509User" must be
provided as "CN=X509User".
If you are using
:atlas:`Atlas-managed certificates </security-add-mongodb-users/#select-an-authentication-method>`
, your username must be prefaced by "CN="
per `RFC 2253 <https://tools.ietf.org/html/rfc2253>`__.
For example, the username "X509User" must be
provided as "CN=X509User".

- id: ldap
name: LDAP
Expand Down
Loading