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

GH-958 - fix for wrong filter deletion #961

Merged
merged 4 commits into from
Aug 13, 2021

Conversation

sbishel
Copy link
Collaborator

@sbishel sbishel commented Aug 12, 2021

Summary

fixes bug where wrong filter was being deleted and last filter couldn't be deleted.

Ticket Link

Fixes #958

@sbishel sbishel requested a review from jespino August 12, 2021 17:58
@sbishel sbishel requested a review from a team as a code owner August 12, 2021 17:58
@sbishel sbishel requested review from hahmadia and removed request for a team August 12, 2021 17:58
@sbishel sbishel added 2: Dev Review Requires review by a core committer CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels Aug 12, 2021
@sbishel sbishel added this to the v0.9 milestone Aug 12, 2021
Copy link
Contributor

@jespino jespino left a comment

Choose a reason for hiding this comment

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

LGTM.

@jespino jespino merged commit 1f36db5 into mattermost-community:main Aug 13, 2021
@mattermod
Copy link
Contributor

Cherry pick is scheduled.

@mattermod
Copy link
Contributor

Error trying doing the automated Cherry picking. Please do this manually

fatal: No such remote 'upstream'

+++ Returning you to the main branch and cleaning up.

jespino added a commit that referenced this pull request Aug 13, 2021
Co-authored-by: Jesús Espino <jespinog@gmail.com>
@jespino jespino added 3: Reviews Complete All reviewers have approved the pull request CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone and removed 2: Dev Review Requires review by a core committer CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels Aug 13, 2021
@sbishel sbishel deleted the gh-958 branch August 13, 2021 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Deleting a filter deletes the wrong one
4 participants