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

Include selected data in exportFunc #775

Merged

Conversation

zwagner86
Copy link
Contributor

@zwagner86 zwagner86 commented Jun 20, 2023

Related Issue

Relate the Github issue with this PR using #

Description

exportFunc receives searched and filtered data and I think it should also receive selected data

Related PRs

List related PRs against other branches:

branch PR
other_pr_production link
other_pr_master link

Impacted Areas in Application

List general components of the application that this PR will affect:

* MTableToolbar

Additional Notes

This is optional, feel free to follow your heart and write here :)

@zwagner86 zwagner86 requested a review from oze4 as a code owner June 20, 2023 05:25
@Domino987
Copy link
Contributor

Thank you

@Domino987
Copy link
Contributor

Seems like some test dont work. Can you check it? @zwagner86

@zwagner86
Copy link
Contributor Author

@Domino987 - I didn't see any test failures on my side, but then I did and attempted to fix. Then my fork was out of date so i synced and realized that my test fixes were not required. So, in short, I am back to not seeing any test failures on my end.

@Domino987 Domino987 merged commit b64ace0 into material-table-core:master Jul 6, 2023
2 of 4 checks passed
@Domino987
Copy link
Contributor

Thank you!

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 this pull request may close these issues.

2 participants