You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Related to #1332 (@AleF83), really cool we have UI for external apps, yet I think we can make it much more usable and clear with a few minor UI tweaks.
Some issues I saw:
The details label is not aligned to secrets label.
The permissions and versions are not relevant (also for creating/editing new apps) and confusing as it won't work.
The App Id should be visible as it's not a secret.
I'd suggested the following changes:
Change the details label to actual app name, and support inline editing (as in key editing). (it'll allow to remove the inner edit page)
Add Id where currently the app name resides.
Remove permissions field in set its data to empty array. (as no app should call directly to authoring, all pass gateway)
Remove version field, it's the type of the manifest, not interesting as well.
Replace buttons for secrets with small +x, the buttons are way too big.
The text was updated successfully, but these errors were encountered:
Related to #1332 (@AleF83), really cool we have UI for external apps, yet I think we can make it much more usable and clear with a few minor UI tweaks.
Some issues I saw:
The details label is not aligned to secrets label.
The permissions and versions are not relevant (also for creating/editing new apps) and confusing as it won't work.
The App Id should be visible as it's not a secret.
I'd suggested the following changes:
Change the details label to actual app name, and support inline editing (as in key editing). (it'll allow to remove the inner edit page)
Add Id where currently the app name resides.
Remove permissions field in set its data to empty array. (as no app should call directly to authoring, all pass gateway)
Remove version field, it's the type of the manifest, not interesting as well.
Replace buttons for secrets with small +x, the buttons are way too big.
The text was updated successfully, but these errors were encountered: