Skip to content

fix(grid): Setup columns when autoGenerate is set after the grid data#16839

Open
mddragnev wants to merge 5 commits intomasterfrom
mdragnev/fix-16830
Open

fix(grid): Setup columns when autoGenerate is set after the grid data#16839
mddragnev wants to merge 5 commits intomasterfrom
mdragnev/fix-16830

Conversation

@mddragnev
Copy link
Member

@mddragnev mddragnev commented Jan 30, 2026

Closes #16830

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them

@mddragnev mddragnev changed the title test(grid): Add a test fix(grid): Setup columns when autoGenerate is set after the grid data Jan 30, 2026
@mddragnev mddragnev marked this pull request as ready for review February 2, 2026 14:04
@mddragnev mddragnev requested a review from damyanpetev February 2, 2026 14:04
@dkamburov dkamburov requested a review from MayaKirova February 3, 2026 14:14
@MayaKirova MayaKirova removed their request for review February 3, 2026 14:39
@MayaKirova MayaKirova self-assigned this Feb 3, 2026
@MayaKirova MayaKirova added 💥 status: in-test PRs currently being tested ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification 💥 status: in-test PRs currently being tested labels Feb 3, 2026
* <igx-grid [data]="Data" [autoGenerate]="true"></igx-grid>
* ```
*/
@WatchChanges()
Copy link
Contributor

Choose a reason for hiding this comment

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

Did we agree to watch for changes, I though we're trying to resolve that specific scenario in react where data is set first and not trying to support the scenario where the dev can change autoGenete? @mddragnev @damyanpetev

Copy link
Member Author

Choose a reason for hiding this comment

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

This is doing exactly that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

grid version: 21.1.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Grids: Setting grid data before autoGenerate of columns after the component is initialized does not render any data

3 participants