Move app settings to a modal and add explanations

[jonatanklosko]

[github-actions]

Uffizzi Preview deployment-25950 was deleted.

[josevalim]

This is looking great, I just want to bikeshed on the panel a bit:

• Is "App" the best name? Maybe "Deploys" is better?

• I also think that "Deployments" should be "Latest version" or "Current version"?

• Also, I think "App sessions" could have its own header, like "Deployments" and maybe call it "Running sessions" or "App sessions" or just "Sessions"?

Thoughts?

[josevalim]

Looks great. I pushed a bunch of nitpicks

[jonatanklosko]

Is "App" the best name? Maybe "Deploys" is better?

We call the feature "apps" in general and we have "Apps" in the home sidebar, so I think "App" gives a clearer indication of what this sidebar is related to.

I also think that "Deployments" should be "Latest version" or "Current version"?

Sounds good, "Current version" works.

Also, I think "App sessions" could have its own header, like "Deployments" and maybe call it "Running sessions" or "App sessions" or just "Sessions"?

A separate header works for me, the wording should be consistent with the apps page:

It could be "sessions", but being explicit doesn't hurt and this makes a clearer distinction between regular sessions and app sessions.

[jonatanklosko]

[josevalim]

So let's go with these headers: "Application", "Latest deploy" (or "Current deploy") and "App sessions". :)

[jonatanklosko]

@josevalim we say "app" everywhere, so for consistency it's better than "application", no?

[jonatanklosko]

Current deploy

Deploy is a verb, not sure if that mattes though :p

[josevalim]

Alright so:

"App", "Latest deployment" (or "Current deployment") and "Running sessions"? I don't want to say "App sessions" because App is implied?

[jonatanklosko]

Sounds good!

[josevalim]

LGTM after the comments have been addressed. :)