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

[data grid] Add Api type param to cell params model #15968

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

k-rajat19
Copy link
Contributor

Closes #15963

@k-rajat19 k-rajat19 changed the title [data grid] Add Api type param to cell params interfaces [data grid] Add Api type param to cell params model Dec 21, 2024
@mui-bot
Copy link

mui-bot commented Dec 21, 2024

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

Generated by 🚫 dangerJS against 52905bc

@k-rajat19
Copy link
Contributor Author

k-rajat19 commented Dec 27, 2024

@michelengelen, could you review this?
Side Note: I'm not reexporting these types for pro and premium packages separately, one can pass pro and premium grid API in Api type param easily.

Copy link
Member

@michelengelen michelengelen left a comment

Choose a reason for hiding this comment

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

LGTM, but I would like a second review from @arminmeh about this.

@zannager zannager added the component: data grid This is the name of the generic UI component, not the React module! label Dec 30, 2024
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
4 participants