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

Getting issues Accessing Nested Properties #158

Closed
MosesWakanyi opened this issue Aug 11, 2020 · 5 comments · Fixed by #159
Closed

Getting issues Accessing Nested Properties #158

MosesWakanyi opened this issue Aug 11, 2020 · 5 comments · Fixed by #159

Comments

@MosesWakanyi
Copy link

Hello,

This property seems not to work now. What has changed?

Product Name
@Gomez
Copy link
Contributor

Gomez commented Aug 14, 2020

Same here, was it working before? Tested with 3.4.x, but still the same.

Here is my code

    <Cell dataKey="node.name" />

data:

          "node": {
            "id": "174692e2-1f76-5fc2-9b78-4b155314c7b2",
            "name": "Accessibility",
            "version": "1.5.0",
            "desc": "Provides multiple accessibilities options to ease your use of nextcloud.",
            "added": false,
            "updated": false
          }
        },
        {
          "node": {
            "id": "b4f5c18e-30cc-5f2e-b0a4-c058f3dd2e3c",
            "name": "Activity",
            "version": "2.12.0",
            "desc": "Provides an activity feed showing your file changes and other interesting things going on in your Nextcloud.",
            "added": false,
            "updated": false
          }
        },

@MosesWakanyi
Copy link
Author

Forced to use this, wondering whether it's efficient!
Cell>
{(rowData, rowIndex) => {
return rowData.node.name;
}}

@simonguo
Copy link
Member

Currently does not support the nested properties.
@MosesWakanyi The solution you use is effective.

<Column width={200}>
  <HeaderCell>Name</HeaderCell>
  <Cell>{rowData => rowData.node.name}</Cell>
</Column>

@Gomez
Copy link
Contributor

Gomez commented Aug 17, 2020

Thanks works. Added a note in the readme: #159

@axtscz
Copy link

axtscz commented Aug 15, 2021

This workaround no longer appears to be working

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants