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

Exporter Excludes Parent Nodes (TreeView) #6819

Closed
TehCupcakes opened this issue Jul 31, 2018 · 2 comments · Fixed by #7129
Closed

Exporter Excludes Parent Nodes (TreeView) #6819

TehCupcakes opened this issue Jul 31, 2018 · 2 comments · Fixed by #7129

Comments

@TehCupcakes
Copy link

The "All data" option for the exporter is insufficient because it does not actually export all data. When combined with TreeView, it presumes that any parent nodes are merely for grouping and categorizing data, and not part of the data itself. It seems to me that there should be another gridOption to use with exporter that lets you include all data, not just leaf nodes. (And in my opinion that should be the default behavior.)

This is regardless of which export format is chosen. It comes down to the getData and getDataSorted functions in ui-grid.exporter.js.

@SanJagDar
Copy link

SanJagDar commented Jan 23, 2019

I am having the same trouble exporting data inside a Tree View. Only leaf nodes get exported and not the parent nodes. Applies across all file formats chosen.

What i also notice is that, 'Export visible data' does indeed export parent nodes plus the child leaf nodes but only those visible on the current page. So, the logic to include parent nodes exists but it does not work across all pages for the 'Export all data' functionality. I can achieve the 'Export all data' functionality by making all data show up on just one page (using pagination) and then use the 'Export visible data'.

So, the code for what i need exists within UI Grid but it only needs to be extended.

@DeanDevlos
Copy link

I would love to see this fixed as well, but thanks to @SanJagDar for the tip about Export visible!

mportuga pushed a commit that referenced this issue Oct 24, 2020
mportuga pushed a commit that referenced this issue Oct 24, 2020
mportuga pushed a commit that referenced this issue Oct 24, 2020
mportuga pushed a commit that referenced this issue Oct 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants