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

Minor adjustments #4168

Merged
merged 1 commit into from
Feb 1, 2019
Merged

Minor adjustments #4168

merged 1 commit into from
Feb 1, 2019

Conversation

emeeks
Copy link
Member

@emeeks emeeks commented Feb 1, 2019

This should resolve #4129

  • Filter value seems to be passing a string now instead of a number (for the numerical filter) so cast to number
  • Return the results of the filtering
  • Update types to reflect that the data object will be passing numbers to the number filter (even though the filter is passing a string)

* Filter value seems to be passing a string now instead of a number (for the numerical filter) so cast to number
* Return the results of the filtering
* Update types to reflect that the data object will be passing numbers to the  number filter (even though the filter is passing a string)
Copy link
Member

@stormpython stormpython left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@BenRussert BenRussert left a comment

Choose a reason for hiding this comment

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

Makes a lot of sense, thanks!

@BenRussert BenRussert merged commit b8b58a4 into master Feb 1, 2019
@rgbkrk rgbkrk deleted the fix-dx-column-filter branch February 1, 2019 16:58
@rgbkrk
Copy link
Member

rgbkrk commented Feb 1, 2019

Cool, let's do another release later today.

@lock lock bot added the outdated workflow: issue should stay closed label May 3, 2019
@lock lock bot locked as resolved and limited conversation to collaborators May 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated workflow: issue should stay closed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Data explorer table filters
4 participants