You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If I have a grid with say the columns [Requested, Settled, Requested, Settled] Only the data from the last two columns is returned.
So for | Requested | Settled | Requested | Settled |
| 10 | 5 | 6 | 8 |
A call to getAgGridData() would return [{"Requested": 6, "Settled": 8}]
In most scenarios where this occurs I imagine the user will have some group headers to differentiate between the columns themselves - so I propose adding the group header names to the column names to get around this.
So for | Total Price | Total Volume |
| Requested | Settled | Requested | Settled |
| 10 | 5 | 6 | 8 |
Any progress on this? We've got the same problem to work around.
If this is proving too complex, perhaps a simple alternative or option for getAgGridData() to return the cells keyed by columnId instead of columnName.
Hey @nealeu and @OVO-Josh! Apologies for the delay in responding, I've been preoccupied with other projects at the moment and haven't had a chance to come back and resolve some of the open issues. I am hoping to be able to do so soon, but in the meantime I am welcoming any PRs and will review ASAP if anyone is able to tackle them before I can.
If I have a grid with say the columns
[Requested, Settled, Requested, Settled]
Only the data from the last two columns is returned.So for | Requested | Settled | Requested | Settled |
| 10 | 5 | 6 | 8 |
A call to
getAgGridData()
would return[{"Requested": 6, "Settled": 8}]
In most scenarios where this occurs I imagine the user will have some group headers to differentiate between the columns themselves - so I propose adding the group header names to the column names to get around this.
So for | Total Price | Total Volume |
| Requested | Settled | Requested | Settled |
| 10 | 5 | 6 | 8 |
We'd get
[{"Total Price Requested": 10, "Total Price Settled": 5, "Total Volume Requested": 6, "Total Volume Settled": 8}]
I currently have a working fix for this locally - happy to put up a PR to add it if wanted?
The text was updated successfully, but these errors were encountered: