-
Notifications
You must be signed in to change notification settings - Fork 900
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
feat(core/managed): Add resource dropdown with links to history + source JSON #7376
feat(core/managed): Add resource dropdown with links to history + source JSON #7376
Conversation
@@ -12,35 +15,75 @@ export interface IManagedResourceDetailsIndicatorProps { | |||
entityTags: IEntityTags[]; | |||
} | |||
|
|||
const logClick = (label: string, resourceId: string) => |
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.
also some usage tracking for fun.
text-shadow: 0px 1px 4px rgba(0, 0, 0, 0.5); | ||
padding-right: 5px; | ||
} | ||
|
||
.ManagedResourceDetailsIndicator .summary-message { | ||
cursor: help; |
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.
nice touch
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 try (sometimes)
d8f1a51 fix(triggers): Protecting from undefined triggers (spinnaker#7377) ad4c144 feat(core/managed): Add resource dropdown with links to history + source JSON (spinnaker#7376) 3665d3e fix(typos): fix a few typos in help messages (spinnaker#7373)
* chore(core): Bump version to 0.0.406 d8f1a51 fix(triggers): Protecting from undefined triggers (#7377) ad4c144 feat(core/managed): Add resource dropdown with links to history + source JSON (#7376) 3665d3e fix(typos): fix a few typos in help messages (#7373) * chore(amazon): Bump version to 0.0.209 373f581 fix(titus): Removing NLB as it is not compatible with Titus (#7317)
* chore(core): Bump version to 0.0.406 d8f1a51 fix(triggers): Protecting from undefined triggers (spinnaker#7377) ad4c144 feat(core/managed): Add resource dropdown with links to history + source JSON (spinnaker#7376) 3665d3e fix(typos): fix a few typos in help messages (spinnaker#7373) * chore(amazon): Bump version to 0.0.209 373f581 fix(titus): Removing NLB as it is not compatible with Titus (spinnaker#7317)
There's a couple bits of data about a resource we expect folks to care about early on: the event history of a resource, and the underlying config/definition for the resource that Keel has on file. This adds a dropdown to managed resources in the infrastructure views that lets folks see both, as new tabs with the raw JSON payload (opting for JSON instead of YAML to start, as we already do that across spinnaker and JSON is easy to use with the API ad-hoc). Design is quick and dirty for now, as we're not sure what we'll need in the coming months.