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

Filter: Unable to change an existing query filter #87

Closed
siddharthlatest opened this issue Jul 13, 2017 · 1 comment
Closed

Filter: Unable to change an existing query filter #87

siddharthlatest opened this issue Jul 13, 2017 · 1 comment
Labels
Milestone

Comments

@siddharthlatest
Copy link
Member

siddharthlatest commented Jul 13, 2017

Dejavu's UI breaks when re-applying the same filter again. Able to replicate on the hosted version at https://opensource.appbase.io/dejavu/.

Steps to replicate:

  1. Applying a "greater than" filter

  2. Removing the filter by clicking on the cross icon

  3. Attempt to re-apply the same filter

The filter does not get applied.

Error log after 3.

Uncaught TypeError: i.props.filterInfo.applyFilter is not a function
    at i.applyFilter (live.js:1)
    at Object.r (live.js:1)
    at a (live.js:1)
    at Object.s [as executeDispatchesInOrder] (live.js:1)
    at d (live.js:1)
    at m (live.js:1)
    at Array.forEach (<anonymous>)
    at r (live.js:1)
    at Object.processEventQueue (live.js:1)
    at r (live.js:1)
@farhan687
Copy link
Contributor

Fixed here 9a80bc9

@siddharthlatest siddharthlatest added this to the v1.0.0 milestone Jul 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants