Fix bug with sorting applied to other colums having using function fields #803
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
codesandbox that shows the issue: https://codesandbox.io/s/clever-fog-2lfgg
@xaksis I'm not 100% sure this is the proper way of fixing this. I'm also not a full fledged js engineer. But the way it's now, using bound functions as fields, can cause multiple issues throughout the app.
I"m using function.name, to translate the bound function to something like:
bound functionName
.This at least fixes my issue. Where when i'm using multiple bound functions for the fields, sorting one will also sort the other.
You might want to look into something a little less hacky? But that would require to refactor parts of the app. And store the field functions in a different format?