Updates wording for codespaces accept permissions flow

Mark Phelps · Mark Phelps · commit 625f3ac14482 · 2022-03-03T15:36:53.000-05:00
diff --git a/pkg/cmd/codespace/create.go b/pkg/cmd/codespace/create.go
@@ -164,7 +164,7 @@ func (a *App) handleAdditionalPermissions(ctx context.Context, createParams *api
 	}
 
 	choices := []string{
-		"Continue in browser to review and authorize additional permissions",
+		"Continue in browser to review and authorize additional permissions (Recommended)",
 		"Continue without authorizing additional permissions",
 	}
 
@@ -190,6 +190,7 @@ func (a *App) handleAdditionalPermissions(ctx context.Context, createParams *api
 
 	// if the user chose to continue in the browser, open the URL
 	if answers.Accept == choices[0] {
+		fmt.Fprintln(a.io.ErrOut, "Please re-run the create request after accepting permissions in the browser.")
 		if err := a.browser.Browse(allowPermissionsURL); err != nil {
 			return nil, fmt.Errorf("error opening browser: %w", err)
 		}

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ func (a App) handleAdditionalPermissions(ctx context.Context, createParams api`
`164`	`164`	`}`
`165`	`165`
`166`	`166`	`choices := []string{`
`167`		`- "Continue in browser to review and authorize additional permissions",`
	`167`	`+ "Continue in browser to review and authorize additional permissions (Recommended)",`
`168`	`168`	`"Continue without authorizing additional permissions",`
`169`	`169`	`}`
`170`	`170`
`@@ -190,6 +190,7 @@ func (a App) handleAdditionalPermissions(ctx context.Context, createParams api`
`190`	`190`
`191`	`191`	`// if the user chose to continue in the browser, open the URL`
`192`	`192`	`if answers.Accept == choices[0] {`
	`193`	`+ fmt.Fprintln(a.io.ErrOut, "Please re-run the create request after accepting permissions in the browser.")`
`193`	`194`	`if err := a.browser.Browse(allowPermissionsURL); err != nil {`
`194`	`195`	`return nil, fmt.Errorf("error opening browser: %w", err)`
`195`	`196`	`}`