Skip to content

Add InstanceID filter to diag webUI #350

Closed
barrett370 wants to merge 4 commits intocschleiden:mainfrom
barrett370:diag-instance-id-filtering
Closed

Add InstanceID filter to diag webUI #350
barrett370 wants to merge 4 commits intocschleiden:mainfrom
barrett370:diag-instance-id-filtering

Conversation

@barrett370
Copy link
Copy Markdown

Small PR to add ability to filter displayed instances in the WebUI, useful if instance Ids follow a specific convention to limit view to workflows related to a specific resource

@cschleiden
Copy link
Copy Markdown
Owner

Do I see that correctly, is that only client-side filtering of the current page?

@barrett370
Copy link
Copy Markdown
Author

Do I see that correctly, is that only client-side filtering of the current page?

yes, just filtering the results that are being displayed. I see that in a recent PR you have fixed pagination to fetch only the instances to be rendered from the database, so perhaps this approach isn't as useful as it only works on a page-by-page basis

@cschleiden
Copy link
Copy Markdown
Owner

Closing this for now. I think the idea still makes sense, but would like to see this more as a search instead of page/page filtering.

@cschleiden cschleiden closed this Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants