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] Rename useGridApiEventHandler to useGridEvent #17159

Merged
merged 6 commits into from
Mar 28, 2025

Conversation

romgrk
Copy link
Contributor

@romgrk romgrk commented Mar 27, 2025

Changelog

Breaking changes

  • The useGridApiEventHandler() hook has been renamed to useGridEvent().
  • The useGridApiOptionHandler() hook has been renamed to useGridEventPriority().

@romgrk romgrk added breaking change component: data grid This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature labels Mar 27, 2025
@romgrk romgrk requested a review from a team March 27, 2025 19:01
@mui-bot
Copy link

mui-bot commented Mar 27, 2025

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 🚢

@cherniavskii cherniavskii changed the title [DataGrid] rename useGridApiEventHandler to useGridEvent [DataGrid] Rename useGridApiEventHandler to useGridEvent Mar 28, 2025
@cherniavskii cherniavskii added this to the 8.0.0 milestone Mar 28, 2025
Co-authored-by: Armin Mehinovic <4390250+arminmeh@users.noreply.github.com>
Signed-off-by: Rom Grk <romgrk@users.noreply.github.com>
@romgrk romgrk enabled auto-merge (squash) March 28, 2025 19:45
@romgrk romgrk merged commit f44a0a6 into mui:master Mar 28, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change component: data grid This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants