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

When using lookup the filtering returns empty results #474

Closed
mastercoks opened this issue Apr 16, 2019 · 6 comments
Closed

When using lookup the filtering returns empty results #474

mastercoks opened this issue Apr 16, 2019 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@mastercoks
Copy link

Describe the bug
When using the filter using lookup, it returns "No records to display"

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://mbrn.github.io/material-table/#/docz-examples-10-example-component-overriding
  2. Select the filter with option 'Şanlıurfa' on the column 'Doğum Yeri'
  3. See 'No records to display'

Expected behavior
Data records with value 'Şanlıurfa' in the 'Doğum Yeri' field should be seen in the table

Screenshots
Print 1
Print 2

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser chrome 73.0.3683.103 64 bits

Additional context
Looking at the code, I checked that it takes the text from the table, and matches the value, so it returns empty.
material-table\dist\utils\data-manager.js
material-table\dist\utils\data-manager.js
Print 4

@mbrn mbrn self-assigned this Apr 16, 2019
@mbrn mbrn added the bug Something isn't working label Apr 16, 2019
@mbrn
Copy link
Owner

mbrn commented Apr 16, 2019

Hi @mastercoks

I will resolve this issue on next release. It will be published in a few days.

@john-pc
Copy link

john-pc commented Apr 29, 2019

@mbrn any updates 🥇 ?

@mbrn
Copy link
Owner

mbrn commented Apr 29, 2019

Hi @john-pc

I am very busy nowadays. But i will start to resolve issues in a few days.

@john-pc
Copy link

john-pc commented Apr 29, 2019

@mbrn I will make a PR

@mbrn
Copy link
Owner

mbrn commented Apr 29, 2019

@john-pc This would be nice. Thanks.

@mbrn
Copy link
Owner

mbrn commented May 2, 2019

Published with 1.34.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants