Skip to content

chore: touch ups to wsproxy UX #8350

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 15 commits into from
Jul 7, 2023
Merged

chore: touch ups to wsproxy UX #8350

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9404e0e
chore: update wording on wsproxy help
Emyrk Jul 6, 2023
4976787
chore: show help if no fields specified in wsproxy edit
Emyrk Jul 6, 2023
f48843d
chore: Add run command example to wsproxy create
Emyrk Jul 6, 2023
5ce6a6f
chore: remove localhost warning
Emyrk Jul 6, 2023
078a3cf
chore: navbar match page title
Emyrk Jul 6, 2023
4b54ae4
chore: Add helper text to latency picker
Emyrk Jul 6, 2023
aac4738
wording
Emyrk Jul 6, 2023
7a073e8
chore: add confirm delete to workspace proxy delete cli
Emyrk Jul 6, 2023
ec2cfb3
chore: add errors + warnings to workspace proxy table
Emyrk Jul 6, 2023
120e281
add -y to unit test
Emyrk Jul 6, 2023
6d28174
Remove unused import
Emyrk Jul 6, 2023
588d07a
Fix import
Emyrk Jul 6, 2023
1cebd2b
remove unused code
Emyrk Jul 6, 2023
3fb4286
fix wording on proxy select text
Emyrk Jul 7, 2023
12f680f
Button on the same line
Emyrk Jul 7, 2023
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
Prev Previous commit
Next Next commit
chore: add confirm delete to workspace proxy delete cli
  • Loading branch information
@Emyrk
Emyrk committed Jul 6, 2023
commit 7a073e8ac0f671df3252d77b0d980980a92a9a25
21 changes: 20 additions & 1 deletion enterprise/cli/workspaceproxy.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -189,13 +189,32 @@ func (r *RootCmd) deleteProxy() *clibase.Cmd {
cmd := &clibase.Cmd{
Use: "delete <name|id>",
Short: "Delete a workspace proxy",
Options: clibase.OptionSet{
cliui.SkipPromptOption(),
},
Middleware: clibase.Chain(
clibase.RequireNArgs(1),
r.InitClient(client),
),
Handler: func(inv *clibase.Invocation) error {
ctx := inv.Context()
err := client.DeleteWorkspaceProxyByName(ctx, inv.Args[0])

wsproxy, err := client.WorkspaceProxyByName(ctx, inv.Args[0])
if err != nil {
return xerrors.Errorf("fetch workspace proxy %q: %w", inv.Args[0], err)
}

// Confirm deletion of the template.
_, err = cliui.Prompt(inv, cliui.PromptOptions{
Text: fmt.Sprintf("Delete this workspace proxy: %s?", cliui.DefaultStyles.Code.Render(wsproxy.DisplayName)),
IsConfirm: true,
Default: cliui.ConfirmNo,
})
if err != nil {
return err
}

err = client.DeleteWorkspaceProxyByName(ctx, inv.Args[0])
if err != nil {
return xerrors.Errorf("delete workspace proxy %q: %w", inv.Args[0], err)
}
Expand Down