- Notifications
You must be signed in to change notification settings - Fork909
feat(cli): add open app <workspace> <app-slug> command#17032
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
70e04ba
toa778d6c
Comparea778d6c
toafbe048
CompareUh oh!
There was an error while loading.Please reload this page.
@@ -54,3 +62,107 @@ func Test_resolveAgentAbsPath(t *testing.T) { | |||
}) | |||
} | |||
} | |||
func Test_buildAppLinkURL(t *testing.T) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
review: I adapted the test cases fromapp.test.ts
. If you have other suggestions for test cases, I'd be happy to add them!
u.RawQuery = q.Encode() | ||
// encodeURIComponent replaces spaces with %20 but url.QueryEscape replaces them with +. | ||
// We replace them with %20 to match the TypeScript implementation. | ||
u.RawQuery = strings.ReplaceAll(u.RawQuery, "+", "%20") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
review: I'm not sure if this is strictly necessary? Did it for completeness' sake.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I'd like to see a slightly better UX, but otherwise this looks great. Nice work!
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Niiice, thanks for improving the UX!
Uh oh!
There was an error while loading.Please reload this page.
0474888
intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Fixes#17009
Adds a CLI command
coder open app <workspace> <app-slug>
that allows opening arbitrarycoder_apps
via the CLI.Users can optionally specify a preferred region for workspace applications.