Skip to content

[Doc] Add details about Datagrid to DataTable migration#10884

Merged
fzaninotto merged 2 commits intomasterfrom
docs/datagrid-codemod-limits
Aug 12, 2025
Merged

[Doc] Add details about Datagrid to DataTable migration#10884
fzaninotto merged 2 commits intomasterfrom
docs/datagrid-codemod-limits

Conversation

@Madeorsk
Copy link
Contributor

@Madeorsk Madeorsk commented Aug 7, 2025

Problem

We have a codemod, but it doesn't do everything. Besides, how to check that it did the right job?

Solution

  • List all the things the codemod does (e.g. replace Datagrid by DataTable)
  • List all the things it doesn’t do and that must be done by hand (e.g. remove DatagridConfigurable)

Additional Checks

  • The PR targets master for a bugfix or a documentation fix, or next for a feature
  • The documentation is up to date

@Madeorsk Madeorsk added the RFR Ready For Review label Aug 7, 2025
@fzaninotto fzaninotto merged commit 6833126 into master Aug 12, 2025
8 of 10 checks passed
@fzaninotto fzaninotto deleted the docs/datagrid-codemod-limits branch August 12, 2025 04:33
@fzaninotto fzaninotto added this to the 5.10.2 milestone Aug 12, 2025
@fzaninotto fzaninotto changed the title Document limits of Datagrid to DataTable codemod [Doc] Add details about Datagrid to DataTable migration Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RFR Ready For Review

Development

Successfully merging this pull request may close these issues.

2 participants