Skip to content

URI handler with support for connection strings#149

Merged
tnaum-ms merged 17 commits into
nextfrom
dev/tnaum/148-activate-url-handler-with-documentdb-connection-details
Jun 13, 2025
Merged

URI handler with support for connection strings#149
tnaum-ms merged 17 commits into
nextfrom
dev/tnaum/148-activate-url-handler-with-documentdb-connection-details

Conversation

@tnaum-ms

Copy link
Copy Markdown
Collaborator

Closes #148

Copilot AI review requested due to automatic review settings June 11, 2025 17:15
@tnaum-ms tnaum-ms linked an issue Jun 11, 2025 that may be closed by this pull request

This comment was marked as outdated.

@tnaum-ms tnaum-ms force-pushed the dev/tnaum/148-activate-url-handler-with-documentdb-connection-details branch from da756b5 to 0f04314 Compare June 13, 2025 09:07
@tnaum-ms tnaum-ms changed the title wip: first steps towards URI handler, support for connection strings URI handler with support for connection strings Jun 13, 2025
@tnaum-ms tnaum-ms requested a review from Copilot June 13, 2025 10:07

This comment was marked as outdated.

@tnaum-ms tnaum-ms requested a review from Copilot June 13, 2025 21:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a URI handler for connection strings by updating configuration keys, localization strings, and enhancing tree data provider methods to support recursive searches. It also updates documentation to include a manual section on constructing URL connections.

  • Updated localization messages and configuration keys for URL handling and confirmation dialogs.
  • Modified tree data provider methods to support optional parameters for recursive search.
  • Expanded documentation with new manuals on URL construction and additional learn-more content.

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/utils/dialogs/getConfirmation.ts Updated confirmation message localization string.
src/tree/connections-view/ConnectionsBranchDataProvider.ts Modified signatures to support optional parameters for recursive search.
src/tree/api/revealConnectionsViewElement.ts Added API to reveal connection elements using updated recursive search.
src/tree/TreeParentCache.ts Improved ancestor search algorithm with updated comments.
src/tree/ExtendedTreeDataProvider.ts Updated interface documentation for recursive search options.
src/extensionVariables.ts Updated configuration key imports and added tree view variable.
src/documentdb/ClustersExtension.ts Updated tree view creation to use the new tree view variable.
package.json Updated configuration keys and their order for confirmation dialogs.
l10n/bundle.l10n.json Updated localized strings to reflect new configuration and messaging.
docs/manual/, docs/learn-more/, docs/index.md Added/updated documentation on URL construction and extended docs.

@tnaum-ms tnaum-ms merged commit 5af155b into next Jun 13, 2025
5 checks passed
@tnaum-ms tnaum-ms deleted the dev/tnaum/148-activate-url-handler-with-documentdb-connection-details branch June 13, 2025 21:20
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.

Activate URL Handler with DocumentDB connection details

2 participants