Skip to content

Conversation

@suratdas
Copy link
Collaborator

Visual-Regression-Tracker/Visual-Regression-Tracker#276
Attached video. Will raise backend pull request if this is approved. Currently, it is not possible to select which test runs it needs to apply to.

ApplyIgnore.mov

@suratdas suratdas requested a review from pashidlos June 21, 2021 22:20
@pashidlos
Copy link
Member

@suratdas sorry for delay
I changed API to support bulk operations
not you could utilise POST test-runs/ignoreAreas/add

@suratdas
Copy link
Collaborator Author

suratdas commented Jul 2, 2021

@suratdas sorry for delay
I changed API to support bulk operations
not you could utilise POST test-runs/ignoreAreas/add

I have included the latest changes now.

@suratdas suratdas requested a review from pashidlos July 2, 2021 20:09
Copy link
Member

@pashidlos pashidlos left a comment

Choose a reason for hiding this comment

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

LGTM

@pashidlos pashidlos merged commit b8333aa into master Jul 3, 2021
@pashidlos pashidlos deleted the apply-ignore-all branch July 3, 2021 13:06
@pashidlos
Copy link
Member

@suratdas thanks for contribution!

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.

3 participants