apps: improve UX on create and show screens #137

cyx · 2021-03-06T00:22:43Z

for create / update / show, we should utilize the key value display.

Similarly, add the hint to guide the user what to do next.

$ auth0 apps create
 Name: my awesome app
 Type: Regular Web Application
 Description: dev tester
Creating application... done

=== cyx application created

  NAME           my awesome app
  TYPE           regular web application
  CLIENT ID      vXAtoaFdhlmtWjpIrjb9AUnrGEAOH2MM
  CLIENT SECRET  <redacted>

 ▸    Quickstarts: https://auth0.com/docs/quickstart/webapp
 ▸    Hint: You might wanna try `auth0 test login --client-id vXAtoaFdhlmtWjpIrjb9AUnrGEAOH2MM`

for create / update / show, we should utilize the key value display. Similarly, add the hint to guide the user what to do next.

cyx · 2021-03-06T01:32:04Z

internal/display/display.go

+			var kvs [][]string
+			for _, pair := range v.KeyValues() {
+				k := pair[0]
+				v := pair[1]
+
+				// NOTE(cyx): We can either nuke it or annotate with `<none>`. For now we're choosing to nuke it.
+				if v != "" {
+					kvs = append(kvs, []string{k, v})
+				}
+			}
+			writeTable(r.ResultWriter, nil, kvs)
 		}


@Widcket worth noting that given we have so many empty values, I went with nuking out the empty ones to start. We can play with alternative approaches to avoiding a list of kv pairs with many null values (such as ) later.

That's a good idea 👍🏻

apps: improve UX on create and show screens

f50e97d

for create / update / show, we should utilize the key value display. Similarly, add the hint to guide the user what to do next.

cyx force-pushed the apps-create-ux-improvements branch from 80bf380 to f50e97d Compare March 6, 2021 01:28

cyx commented Mar 6, 2021

View reviewed changes

cyx merged commit 666d061 into main Mar 6, 2021

cyx deleted the apps-create-ux-improvements branch March 6, 2021 02:13

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

apps: improve UX on create and show screens #137

apps: improve UX on create and show screens #137

cyx commented Mar 6, 2021

cyx Mar 6, 2021

Widcket Mar 6, 2021

apps: improve UX on create and show screens #137

apps: improve UX on create and show screens #137

Conversation

cyx commented Mar 6, 2021

cyx Mar 6, 2021

Choose a reason for hiding this comment

Widcket Mar 6, 2021

Choose a reason for hiding this comment