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

Unable to add placeholders for range filters #420

Merged
merged 2 commits into from Nov 17, 2016

Conversation

2 participants
@PingusPepan
Copy link
Contributor

PingusPepan commented Nov 17, 2016

This quick fix fixes #419
The code handling placeholders for ranges could use a rewrite though

@juniwalk juniwalk merged commit c6463ea into contributte:master Nov 17, 2016

2 checks passed

Scrutinizer 1 new issues, 1 updated code elements
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@juniwalk

This comment has been minimized.

Copy link
Collaborator

juniwalk commented Nov 17, 2016

Thanks man!

@PingusPepan PingusPepan deleted the PingusPepan:patch-2 branch Nov 17, 2016

paveljanda added a commit that referenced this pull request Nov 20, 2016

Merge branch 'master' of https://github.com/ublaboo/datagrid
* 'master' of https://github.com/ublaboo/datagrid:
  Unable to add placeholders for range filters; Closes #420
  Added scrutinizer configuration for code coverage; See #417
  Improved tests with unix compatibility; Closes #417
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.