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.

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

[data grid] Remove the unused legacy editing API event rowEditCommit #12071

Closed
MBilalShafi opened this issue Feb 15, 2024 · 1 comment · Fixed by #12073
Closed

[data grid] Remove the unused legacy editing API event rowEditCommit #12071

MBilalShafi opened this issue Feb 15, 2024 · 1 comment · Fixed by #12073
Labels
breaking change component: data grid This is the name of the generic UI component, not the React module! needs cherry-pick The PR should be cherry-picked to master after merge v7.x

Comments

@MBilalShafi
Copy link
Member

MBilalShafi commented Feb 15, 2024 •
edited
Loading

Event removal missed during #6016

More context: #11887 (comment)

Search keywords:
@MBilalShafi MBilalShafi added breaking change component: data grid This is the name of the generic UI component, not the React module! v7.x labels Feb 15, 2024
@MBilalShafi MBilalShafi mentioned this issue Feb 15, 2024
Closed
@MBilalShafi MBilalShafi added the needs cherry-pick The PR should be cherry-picked to master after merge label Feb 15, 2024
@MBilalShafi MBilalShafi mentioned this issue Feb 15, 2024
Merged
@github-project-automation github-project-automation bot moved this to 🆕 Needs refinement in MUI X Data Grid Feb 15, 2024
@MBilalShafi MBilalShafi moved this from 🆕 Needs refinement to 🏗 In progress in MUI X Data Grid Feb 15, 2024
@github-actions GitHub Actions
Copy link

⚠️ This issue has been closed.
If you have a similar problem, please open a new issue and provide details about your specific problem.
If you can provide additional information related to this topic that could help future readers, please feel free to leave a comment.

@DanailH DanailH moved this from 🏗 In progress to ✅ Done in MUI X Data Grid Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change component: data grid This is the name of the generic UI component, not the React module! needs cherry-pick The PR should be cherry-picked to master after merge v7.x
Projects
MUI X Data Grid
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[DataGrid] Remove legacy editing API event: rowEditCommit MBilalShafi/mui-x
1 participant
@MBilalShafi