- Notifications
You must be signed in to change notification settings - Fork24
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 changedLines changed: 5 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + |
Lines changed: 3 additions & 3 deletions
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)