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
[vtadmin-web] Add hooks + skeleton view for workflows #7762
[vtadmin-web] Add hooks + skeleton view for workflows #7762
Conversation
Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
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.
one non-blocking comment, but lg2m!
<tr key={idx}> | ||
<td>{workflow?.name}</td> | ||
<td>{cluster?.name}</td> | ||
<td>{workflow?.source?.keyspace || '-'}</td> |
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 wonder if -
is the right placeholder here, because in vitess that could be confused with the 00-00 shard
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.
Ohhh good idea, thanks! I updated it to "n/a" for now, which is straightforward but also lame. I'll get Shannon's thoughts on a better placeholder this week as we go over the rest of the UI.
east: { | ||
workflows: [ | ||
{ | ||
workflow: { name: 'one-goes-east' }, |
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.
🤌 (cmon github, how is this not in your emoji set yet???)
Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
Addressed feedback (thanks!) and also had a commit with actually correct sort order that I hadn't pushed up (sigh). |
Description
/api/workflows
endpoint/workflows
view as a placeholder for later.Related Issue(s)
N/A
Checklist
Deployment Notes
N/A
Impacted Areas in Vitess
Components that this PR will affect: