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

Add `loadData` support for Nested Rows. #6339

Closed
jansiegel opened this issue Oct 15, 2019 · 2 comments
Closed

Add `loadData` support for Nested Rows. #6339

jansiegel opened this issue Oct 15, 2019 · 2 comments

Comments

@jansiegel
Copy link
Member

@jansiegel jansiegel commented Oct 15, 2019

Description

Issue #3914 pointed to a problem with loadData not being covered by the Nested Rows plugin.

@frankie567 made a pull request (#6015) that adds the loadData support to the plugin, which fixes the problem with replacing data with another dataset, but not loading data when it was previously defined as an empty array.

Demo

@aninde

This comment has been minimized.

Copy link

@aninde aninde commented Oct 15, 2019

The cases from #3914 will be fixed by a new index mapper in 8.0.0-rc1 (14.09.19)

Demo
Case 1
still fixed in v. 8.0.0-rc1 https://jsfiddle.net/aninde/hLf2nw7v/
v. 7.2.0 fixed: https://jsfiddle.net/aninde/1dqceLwj/

Case 2
still fixed in v. 8.0.0-rc1https://jsfiddle.net/aninde/rgdop7yn/
v. 7.2.0 not fixed: https://jsfiddle.net/aninde/5a109gso/

Data and headers update and renders right after clicking on a button.
Screenshot 2019-10-15 at 11 56 40

@aninde

This comment has been minimized.

Copy link

@aninde aninde commented Nov 15, 2019

Case 2 is still fixed by 8.0.0-beta1 https://jsfiddle.net/aninde/ysmn10xu/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.