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

Implement filters for Workflow Runs #12123

Conversation

danidoni
Copy link
Contributor

@danidoni danidoni commented Jan 24, 2022

This PR implements several filters to help you narrow down your search
for a specific workflow run.

It filters by workflow run status (success, running, failure) and by
source event (pull request, push, both GitHub and GitLab).

image

TODO

  • Finish specs for the new WorkflowRunFilterComponent
  • Write specs for WorkflowRunFilterLinkComponent
  • Write spec for WorkflowRunsFinder

@github-actions github-actions bot added the Frontend Things related to the OBS RoR app label Jan 24, 2022
@danidoni danidoni changed the title WIP Implement filters for Workflow Runs Jan 24, 2022
@danidoni danidoni force-pushed the implement-filters-for-a-workflow-run-index-page branch 8 times, most recently from 0201ce8 to 5172d18 Compare January 24, 2022 12:02
@danidoni
Copy link
Contributor Author

@danidoni danidoni force-pushed the implement-filters-for-a-workflow-run-index-page branch 7 times, most recently from 4d76009 to 7160191 Compare January 24, 2022 15:02
@danidoni danidoni marked this pull request as ready for review January 24, 2022 15:03
@dmarcoux
Copy link
Contributor

@krauselukas krauselukas force-pushed the implement-filters-for-a-workflow-run-index-page branch 2 times, most recently from a0dc4ed to 935fa28 Compare February 2, 2022 15:21
@krauselukas krauselukas force-pushed the implement-filters-for-a-workflow-run-index-page branch 5 times, most recently from f69a987 to 423286f Compare February 3, 2022 15:28
@danidoni
Copy link
Contributor Author

danidoni commented Feb 3, 2022

LGTM!

@krauselukas krauselukas force-pushed the implement-filters-for-a-workflow-run-index-page branch from 423286f to 8b4a640 Compare February 3, 2022 15:36
src/api/app/policies/workflow_run_policy.rb Outdated Show resolved Hide resolved
@krauselukas krauselukas force-pushed the implement-filters-for-a-workflow-run-index-page branch from 8b4a640 to 3378bde Compare February 3, 2022 17:05
@saraycp saraycp self-requested a review February 4, 2022 13:02
@saraycp
Copy link
Contributor

saraycp commented Feb 4, 2022

I suggest a correction for the commit description "Move hook_event logic from workflow_run_row view component to model" s/the model it self then the view component./the model itself than the view component.

@hennevogel hennevogel dismissed their stale review February 4, 2022 14:59

feedback got addressed

@krauselukas krauselukas force-pushed the implement-filters-for-a-workflow-run-index-page branch 2 times, most recently from 1f44804 to b865037 Compare February 4, 2022 15:21
Copy link
Contributor

@saraycp saraycp left a comment

Choose a reason for hiding this comment

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

LGTM!

krauselukas and others added 5 commits February 4, 2022 16:28
This logic can also be used in the workflow runs finder class and
better fits in the model itself than the view component.

Co-authored-by: Daniel Donisa <daniel.donisa@suse.com>
This PR implements several filters to help you narrow down your search
for a specific workflow run.

It filters by workflow run status (success, running, failure) and by
source event (pull request, push, both GitHub and GitLab).

Co-authored-by: Daniel Donisa <daniel.donisa@suse.com>
@krauselukas krauselukas force-pushed the implement-filters-for-a-workflow-run-index-page branch from b865037 to 9d11375 Compare February 4, 2022 15:28
@krauselukas krauselukas merged commit ea402e1 into openSUSE:master Feb 4, 2022
@danidoni danidoni deleted the implement-filters-for-a-workflow-run-index-page branch February 7, 2022 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Things related to the OBS RoR app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants