Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 21, 2025

Addresses feedback from #398 to improve user experience when removeConnection is invoked without a connection selection.

Changes

  • Added ext.outputChannel.warn() when connectionsToDelete.length === 0 to inform users instead of silently returning
  • Updated localization bundle with new warning string
if (connectionsToDelete.length === 0) {
    ext.outputChannel.warn(l10n.t('No connections selected to remove.'));
    return;
}

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix connection removal feature based on feedback Add warning for removeConnection invoked with no selection Nov 21, 2025
Copilot AI requested a review from tnaum-ms November 21, 2025 18:07
@tnaum-ms tnaum-ms marked this pull request as ready for review November 24, 2025 09:57
@tnaum-ms tnaum-ms requested a review from a team as a code owner November 24, 2025 09:57
@tnaum-ms tnaum-ms merged commit 2570556 into dev/tnaum/connection-multi-delete Nov 24, 2025
1 check passed
@tnaum-ms tnaum-ms deleted the copilot/sub-pr-398-again branch November 24, 2025 09:57
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.

2 participants