- Notifications
You must be signed in to change notification settings - Fork35
Commit9d80d67
authored
fix: handle agent parameter in URIs (#538)
This fixes a bug where the `agent` query parameter on the extension URI was ignored.We were previously doing:```tsvscode.commands.executeCommand("coder.openDevContainer",workspaceOwner,workspaceName,workspaceAgent,devContainerName,devContainerFolder,);```where `args[2]` was the agent name, but we were discarding it:```tsconst workspaceOwner = args[0] as string;const workspaceName = args[1] as string;const workspaceAgent = undefined; // args[2] is reserved, but we do not support multiple agents yet.const devContainerName = args[3] as string;const devContainerFolder = args[4] as string;```The same was true for the `coder.open` command. Presumably due to the comment saying multiple agents aren't supported, which hasn't been true for years.1 parentd1289c6 commit9d80d67
2 files changed
+8
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
600 | 600 | | |
601 | 601 | | |
602 | 602 | | |
603 | | - | |
| 603 | + | |
604 | 604 | | |
605 | 605 | | |
606 | 606 | | |
| |||
628 | 628 | | |
629 | 629 | | |
630 | 630 | | |
631 | | - | |
| 631 | + | |
632 | 632 | | |
633 | 633 | | |
634 | 634 | | |
| |||
748 | 748 | | |
749 | 749 | | |
750 | 750 | | |
751 | | - | |
| 751 | + | |
752 | 752 | | |
753 | 753 | | |
754 | 754 | | |
| |||
0 commit comments
Comments
(0)