Skip to content

Conversation

@MayaKirova
Copy link
Contributor

@MayaKirova MayaKirova commented Sep 12, 2024

Closes #13659
Closes #14896

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

@MayaKirova MayaKirova marked this pull request as ready for review September 13, 2024 13:41
@MayaKirova MayaKirova added the ❌ status: awaiting-test PRs awaiting manual verification label Sep 13, 2024
@MayaKirova
Copy link
Contributor Author

Note to self: For scenarios where you have moved a child column inside a group column, sort keys so that they still match.

@dkamburov dkamburov 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 Oct 18, 2024
damyanpetev
damyanpetev previously approved these changes Oct 21, 2024
Copy link
Member

@damyanpetev damyanpetev left a comment

Choose a reason for hiding this comment

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

Thinking this is a neat enhancement, could do with with being documented in the changelog. Otherwise, LGTM.

dkamburov
dkamburov previously approved these changes Oct 21, 2024
@dkamburov
Copy link
Contributor

@MayaKirova I couldn't merge it, can you please resolve the conflict?

@dkamburov dkamburov merged commit 8c0d755 into master Oct 21, 2024
@dkamburov dkamburov deleted the mkirova/feat-state-reuse-columns branch October 21, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

grid: state-persistence version: 18.2.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-use column instances for state persistence [ELEMENTS] Setting template on a column on columnInit event in WC causes an error

5 participants