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

Fixes for useExtended in useTable plus small bug #1279

Merged
merged 2 commits into from Feb 27, 2019

Conversation

Projects
None yet
3 participants
@gary-menzel
Copy link
Collaborator

gary-menzel commented Feb 23, 2019

I wanted to get useExtended going but it was failing so fixed up the problems where the row specific hooks were not being applied and the path string for the key was not unique.

There was also a bug (which I saw in the useGroupBy code).

[PS - sorry for the incorrect username - it is fixed in my local .git/config now]

ITAdminRMX and others added some commits Feb 23, 2019

undefined error in GroupBy
row specific hooks and correct pathing for nested rows

@tannerlinsley tannerlinsley merged commit a9971e6 into master Feb 27, 2019

1 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details
deploy/netlify Deploy preview ready!
Details
@tannerlinsley

This comment has been minimized.

Copy link
Owner

tannerlinsley commented Feb 27, 2019

Nice work Gary!

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.