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

[component] Custom Mui Data Table component #155

Closed

Conversation

sudhanshutech
Copy link
Member

@sudhanshutech sudhanshutech commented Oct 10, 2023

Notes for Reviewers

This PR fixes #119

I will provide a proper readme also in a while

Signed commits

  • Yes, I signed my commits.

theme?: object;
}

const ResponsiveDataTable: React.FC<ResponsiveDataTableProps<unknown>> = ({
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nebula-aac i think unknown is not a right type, can you provide some guidance in here

@sudhanshutech sudhanshutech added the pr/do-not-merge PRs not ready to be merged label Oct 10, 2023
@sudhanshutech sudhanshutech changed the title Custom Mui Data Table component [component] Custom Mui Data Table component Oct 10, 2023
@sudhanshutech
Copy link
Member Author

@nebula-aac configuring the table is making things hard. I am not able to import the data table intenaly

@nebula-aac
Copy link
Contributor

@leecalcote @sudhanshutech I have a fix for this but I don't have enough time to review it right now. I will provide my feedback this weekend on this PR.

@nebula-aac
Copy link
Contributor

closed as this is already merged in on a different PR

@nebula-aac nebula-aac closed this Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/components pr/do-not-merge PRs not ready to be merged
Development

Successfully merging this pull request may close these issues.

[components] Create data tables component
2 participants