-
Notifications
You must be signed in to change notification settings - Fork 782
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
Data get sorted locally even with remote sorting #1737
Comments
@jfremy I'll check it soon |
However, in the block of |
I've also been stuck on this issue for the past two days and only after reading this post have finally fixed it. |
sure, this will be fixed on next release. |
Thank you. When will the next release occur? |
We are also stuck due to this bug and waiting for the next release. |
11/10, I can promise it, sorry about this mistake. |
Released on |
@AllenFang I have updated to v4.1.2. Now I am getting blank table with message: "There is no data to display" even though I have got data from the server. Could you please check? |
Yes, I'm seeing the same. |
@meet-asite @AllenFang I am getting blank table too, after updated to v4.1.2 just now. 😅 |
I'm shame on this mistake, fixed it again and released on |
@AllenFang To avoid such situations in future for react-bootstrap-table2, do you think we can have e2e tests that runs after every merge to master? |
Something we can done for unit test, e2e test I will consider, but if we have more stronger unit tests in |
I think something went wrong with the 4.1.3 release - looking at the compiled code in lib in the released package, it doesn't contain the fix you added and on the page the table still shows only the "no data" message in the table. |
I am still getting blank table with V4.1.3 |
try again on |
Tables are displaying data properly now. :) |
Thank you @AllenFang remote sorting is working properly now. |
Thanks you guys. it's sorry that I make some mistake when released. :P |
react-bootstrap-table/src/BootstrapTable.js
Line 281 in 97171e1
When using a remote data source and remote sorting, react bootstrap table still sorts the data locally even though it should not.
I think the bug is located in BootstrapTable.js line 281 (in componentWillReceiveProps) and should probably be something like this:
I'm not completely familiar with that part of the code so the fix proposal might not be the right one.
I think the previous condition block (the
if (this.store.isOnFilter) { ... }
) probably need the same kind of logic checking for support of Const.REMOTE_FILTER.The text was updated successfully, but these errors were encountered: