-
Notifications
You must be signed in to change notification settings - Fork 2
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
App Details page: API Keys Section #98
App Details page: API Keys Section #98
Conversation
Signed-off-by: Nicholas Bucher <[email protected]>
Issues linked to changelog: |
Signed-off-by: Nicholas Bucher <[email protected]>
Signed-off-by: Nicholas Bucher <[email protected]>
Signed-off-by: Nicholas Bucher <[email protected]>
Signed-off-by: Nicholas Bucher <[email protected]>
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.
Looks pretty straight-forward and good to me
@@ -108,6 +108,16 @@ export type App = { | |||
teamId: string; | |||
}; | |||
|
|||
export type ApiKey = { |
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 thought we already had this. Is this new or bringing it back?
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.
Yeah it's a throwback 😄 - we had this for the Gloo Mesh Gateway portal server.
With the new implementation of the Gloo Gateway portal server (the version we're moving to with these apps + teams features), it's slightly different. Luckily the code for it was mostly there, and all tested before, so I know that certain features like copyToClipboard
should work across browsers.
This PR:
Screen.Recording.2024-09-11.at.3.48.11.PM.mov
BOT NOTES:
resolves https://github.com/solo-io/solo-projects/issues/6881