Skip to content
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

Add deletion to Workflow List page table #90

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

ohltyler
Copy link
Member

@ohltyler ohltyler commented Feb 29, 2024

Description

This PR adds a delete action to the table to delete the corresponding workflow, and a few other minor improvements to the table. Specifically:

  • adds an actions column with delete functionality
  • finalizes Delete Workflow API response processing and hooks it into the redux store
  • edge case handling of missing workflow state doc
  • adds loading state by using the loading redux field to display a spinner when fetching workflows
  • refreshes list of workflows when navigating back to the 'manage' tab, which is the workflow list page

Demo video:

demo-229.webm

Issues Resolved

Resolves #70
Resolves #65

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Tyler Ohlsen <[email protected]>
@ohltyler ohltyler merged commit 7043c11 into opensearch-project:main Feb 29, 2024
8 of 9 checks passed
@ohltyler ohltyler deleted the add-delete branch February 29, 2024 23:23
opensearch-trigger-bot bot pushed a commit that referenced this pull request Feb 29, 2024
Signed-off-by: Tyler Ohlsen <[email protected]>
(cherry picked from commit 7043c11)
ohltyler added a commit that referenced this pull request Feb 29, 2024
Signed-off-by: Tyler Ohlsen <[email protected]>
(cherry picked from commit 7043c11)

Co-authored-by: Tyler Ohlsen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Workflow list] Integrate with backend APIs [META] Workflow list page
3 participants