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

[DataGrid] follow warning message guideline for autoPageSize and autoHeight #11585

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

Sboonny
Copy link
Contributor

@Sboonny Sboonny commented Jan 5, 2024

the warning doesn't follow the guideline, so I updated it. More context in #11554 (comment).

@mui-bot
Copy link

mui-bot commented Jan 5, 2024

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

Generated by 🚫 dangerJS against b473c3b

@romgrk romgrk added the component: data grid This is the name of the generic UI component, not the React module! label Jan 5, 2024
@romgrk romgrk merged commit 0ed7168 into mui:next Jan 5, 2024
17 of 18 checks passed
@Sboonny Sboonny deleted the refactor/follow-warning-message-guideline branch January 5, 2024 14:13
@romgrk romgrk mentioned this pull request Jan 5, 2024
1 task
@alexfauquette
Copy link
Member

Hi, just for info, chainProptypes is not well supported after bumping the monorepo #11303. So for now, those update will be ignored the time to either fix the parsing script, or move those warning somewhere else

flaviendelangle added a commit to flaviendelangle/mui-x that referenced this pull request Jan 5, 2024
@Sboonny
Copy link
Contributor Author

Sboonny commented Jan 5, 2024

No worries, and thank you for context, I have subscribed to bump monorepo PR. When it's merged, I will try to figure out a way to implement the error back in, if it's open for contribution.

@alexfauquette
Copy link
Member

@cherniavskii mentioning just to avoid duplicate work ⬆️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants