Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

fix(#125): emit remove filter #206

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

cmyip
Copy link

@cmyip cmyip commented Sep 22, 2019

🐛 Bug Fix
Fixes issue #125 , links to apache/superset#8273

@cmyip cmyip requested a review from a team as a code owner September 22, 2019 00:29
@netlify
Copy link

netlify bot commented Sep 22, 2019

Deploy preview for superset-ui-plugins ready!

Built with commit 7b7ced9

https://deploy-preview-206--superset-ui-plugins.netlify.com

@codecov
Copy link

codecov bot commented Sep 22, 2019

Codecov Report

Merging #206 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #206   +/-   ##
=======================================
  Coverage   34.25%   34.25%           
=======================================
  Files          12       12           
  Lines         216      216           
  Branches       21       21           
=======================================
  Hits           74       74           
  Misses        132      132           
  Partials       10       10

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 0b2cf90...7b7ced9. Read the comment docs.

@kristw
Copy link
Collaborator

kristw commented Sep 26, 2019

Have you verified that this works as intended? Could you add some explanation of the behaviors before/after?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants