Skip to content

[DataGrid] Skip calling fetchRows() when strategy is not initialized #19728

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Sep 30, 2025
Merged

[DataGrid] Skip calling fetchRows() when strategy is not initialized #19728

merged 6 commits into from
Sep 30, 2025

Conversation

MBilalShafi
Copy link
Member

Fixes #19650

@MBilalShafi MBilalShafi self-assigned this Sep 26, 2025
@MBilalShafi MBilalShafi added type: bug It doesn't behave as expected. scope: data grid Changes related to the data grid. plan: Pro Impact at least one Pro user. feature: Server integration Better integration with backends, e.g. data source labels Sep 26, 2025
@MBilalShafi MBilalShafi mentioned this pull request Sep 26, 2025
Closed
@mui-bot
Copy link

mui-bot commented Sep 26, 2025 •
edited
Loading

Deploy preview: https://deploy-preview-19728--material-ui-x.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/x-data-grid 🔺+99B(+0.03%) 🔺+19B(+0.02%)
@mui/x-data-grid-pro 🔺+99B(+0.02%) 🔺+27B(+0.02%)
@mui/x-data-grid-premium 🔺+99B(+0.02%) 🔺+21B(+0.01%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 0B(0.00%)
@mui/x-charts-premium 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against 158943a

@MBilalShafi MBilalShafi requested review from a team September 29, 2025 10:33
@MBilalShafi MBilalShafi enabled auto-merge (squash) September 30, 2025 03:57
@MBilalShafi MBilalShafi merged commit b995c97 into mui:master Sep 30, 2025
21 checks passed
@arminmeh arminmeh mentioned this pull request Sep 30, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arminmeh arminmeh arminmeh approved these changes

@michelengelen michelengelen michelengelen approved these changes

Assignees

@MBilalShafi MBilalShafi

Labels
feature: Server integration Better integration with backends, e.g. data source plan: Pro Impact at least one Pro user. scope: data grid Changes related to the data grid. type: bug It doesn't behave as expected.
Projects
None yet
Milestone
No milestone
4 participants
@MBilalShafi @mui-bot @arminmeh @michelengelen