Skip to content

[vtadmin-web] Add a hasty filter input to the /schemas view#7779

Merged
ajm188 merged 1 commit intovitessio:masterfrom
tinyspeck:sarabee-vtadmin-web-schema-filtering-badly
Apr 1, 2021
Merged

[vtadmin-web] Add a hasty filter input to the /schemas view#7779
ajm188 merged 1 commit intovitessio:masterfrom
tinyspeck:sarabee-vtadmin-web-schema-filtering-badly

Conversation

@doeg
Copy link
Contributor

@doeg doeg commented Apr 1, 2021

Signed-off-by: Sara Bee 855595+doeg@users.noreply.github.com

Description

A dumb little PR that adds a filter input to the Schemas view, as we have on the Tablets view.

"Hasty" because this filter input should be extracted into its own component (since we'll use it all over the place) BUT I am impatient and wanted something rudimentary right now as I'm adding table sizes + row counts. :')

Screen Shot 2021-04-01 at 2 06 54 PM

Screen Shot 2021-04-01 at 2 06 59 PM

Screen Shot 2021-04-01 at 2 07 05 PM

Related Issue(s)

N/A

Checklist

  • Should this PR be backported? No
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

N/A

Impacted Areas in Vitess

Components that this PR will affect:

  • Query Serving
  • VReplication
  • Cluster Management
  • Build/CI
  • VTAdmin

Signed-off-by: Sara Bee <855595+doeg@users.noreply.github.com>
@doeg doeg requested review from ajm188, rafael and rohit-nayak-ps April 1, 2021 18:08
Copy link
Contributor

@ajm188 ajm188 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lg2m!

@ajm188 ajm188 merged commit 374ff57 into vitessio:master Apr 1, 2021
@askdba askdba added the Component: VTAdmin VTadmin interface label Apr 3, 2021
@askdba askdba added this to the v10.0 milestone Apr 3, 2021
@doeg doeg deleted the sarabee-vtadmin-web-schema-filtering-badly branch November 1, 2021 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: VTAdmin VTadmin interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants