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

feat: Allow to archive workflow from flyteconsole #361

Merged
merged 13 commits into from
Apr 11, 2022
Merged

Conversation

ursucarina
Copy link
Contributor

@ursucarina ursucarina commented Apr 8, 2022

Signed-off-by: Carina Ursu [email protected]

New feature: Archive/Unarchive workflows

Archive Button: https://share.getcloudapp.com/d5u9nvpy
Archive Actions: https://share.getcloudapp.com/p9u6Q7r2

TL;DR

Allows to filter workflows by archive state in the console, and for individual workflows to be archived/unarchived

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

Added WorkflowExecutionState enum and calls to NamedEntityUpdate
Added notistack to support error and success notifications
Ensured that items could be archived and unarchived from "workflow"
Updates storybook to observe all recent changes
Added "localization" string support

Tracking Issue

adds feature for https://github.com/flyteorg/flyteconsole/issues/320

@ursucarina ursucarina requested review from anrusina, olga-union, a team and jsonporter and removed request for a team April 8, 2022 16:41
@welcome
Copy link

welcome bot commented Apr 8, 2022

Thank you for opening this pull request! 🙌

These tips will help get your PR across the finish line:

  • Most of the repos have a PR template; if not, fill it out to the best of your knowledge.
  • Sign off your commits (Reference: DCO Guide).

@ursucarina ursucarina marked this pull request as draft April 8, 2022 16:42
@codecov
Copy link

codecov bot commented Apr 8, 2022

Codecov Report

Merging #361 (c6f0d53) into master (5842753) will increase coverage by 0.42%.
The diff coverage is 69.66%.

@@            Coverage Diff             @@
##           master     #361      +/-   ##
==========================================
+ Coverage   66.01%   66.43%   +0.42%     
==========================================
  Files         402      405       +3     
  Lines        8965     9045      +80     
  Branches     1580     1594      +14     
==========================================
+ Hits         5918     6009      +91     
+ Misses       3047     3036      -11     
Impacted Files Coverage Δ
src/models/Workflow/api.ts 72.72% <57.14%> (-27.28%) ⬇️
...components/Workflow/SearchableWorkflowNameList.tsx 72.26% <58.62%> (+12.89%) ⬆️
...Executions/Tables/WorkflowExecutionTable/cells.tsx 83.60% <100.00%> (ø)
src/components/Project/ProjectWorkflows.tsx 100.00% <100.00%> (ø)
...s/Workflow/filters/useWorkflowShowArchivedState.ts 100.00% <100.00%> (ø)
src/components/Workflow/useWorkflowInfoList.ts 100.00% <100.00%> (ø)
src/components/Workflow/utils.ts 100.00% <100.00%> (ø)
src/models/Workflow/enums.ts 100.00% <100.00%> (ø)
src/mocks/createAdminServer.ts 91.97% <0.00%> (+0.61%) ⬆️
...xecutions/Tables/WorkflowExecutionTable/strings.ts 53.84% <0.00%> (+7.69%) ⬆️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4721e84...c6f0d53. Read the comment docs.

Copy link
Contributor

@anrusina anrusina left a comment

Choose a reason for hiding this comment

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

Good job overall, left few comments mainly about styling

src/components/Project/ProjectWorkflows.tsx Outdated Show resolved Hide resolved
src/components/Project/ProjectWorkflows.tsx Outdated Show resolved Hide resolved
src/components/Workflow/SearchableWorkflowNameList.tsx Outdated Show resolved Hide resolved
src/components/Workflow/SearchableWorkflowNameList.tsx Outdated Show resolved Hide resolved
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
@ursucarina ursucarina marked this pull request as ready for review April 11, 2022 19:06
Signed-off-by: Carina Ursu <[email protected]>
@ursucarina ursucarina changed the title feat: archive workflows base feat: Allow to archive workflow from flyteconsole Apr 11, 2022
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
olga-union
olga-union previously approved these changes Apr 11, 2022
Copy link
Contributor

@olga-union olga-union left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM!

Copy link
Contributor

@anrusina anrusina left a comment

Choose a reason for hiding this comment

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

Mainly lgtm, need some updates in description and button clarifications.

src/components/Workflow/SearchableWorkflowNameList.tsx Outdated Show resolved Hide resolved
src/components/Project/test/ProjectWorkflows.test.tsx Outdated Show resolved Hide resolved
src/components/Workflow/SearchableWorkflowNameList.tsx Outdated Show resolved Hide resolved
src/components/Workflow/SearchableWorkflowNameList.tsx Outdated Show resolved Hide resolved
src/components/Workflow/utils.ts Show resolved Hide resolved
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Copy link
Contributor

@anrusina anrusina left a comment

Choose a reason for hiding this comment

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

LGTM, Great Job

🚀 🥳

@anrusina anrusina merged commit 1ed28c4 into master Apr 11, 2022
@welcome
Copy link

welcome bot commented Apr 11, 2022

Congrats on merging your first pull request! 🎉

@flyte-bot
Copy link
Collaborator

🎉 This PR is included in version 0.51.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

4 participants