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

Deprecate rowID and primaryKey in grid interfaces #13427

Merged
merged 104 commits into from
Feb 5, 2024

Conversation

ddincheva
Copy link
Contributor

Closes #10617

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

@ddincheva ddincheva self-assigned this Aug 31, 2023
@ddincheva ddincheva added version: 16.1.x 🛠️ status: in-development Issues and PRs with active development on them grid labels Aug 31, 2023
@ddincheva ddincheva marked this pull request as ready for review September 12, 2023 06:17
@ddincheva ddincheva removed the 🛠️ status: in-development Issues and PRs with active development on them label Sep 12, 2023
@hanastasov hanastasov removed the request for review from dobromirts December 16, 2023 17:03
@kdinev
Copy link
Member

kdinev commented Jan 9, 2024

@damyanpetev Reminder :)

@hanastasov
Copy link
Contributor

@ddincheva will shortly resolve the conflicts.

@kdinev
Copy link
Member

kdinev commented Jan 15, 2024

@damyanpetev

@@ -18,6 +18,7 @@ import { NgTemplateOutlet, NgIf, NgFor, NgStyle, NgClass } from '@angular/common
})
export class IgxGridRowComponent extends IgxRowDirective {

// TODO, is this public?
Copy link
Member

Choose a reason for hiding this comment

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

It isn't :)

CHANGELOG.md Outdated
@@ -22,6 +22,15 @@ All notable changes for each version of this project will be documented in this
- Unified logic for vertical and horizontal virtualization such as - caching, updating, max browser size exceeding.
- Addded new method - `addScroll` that can shift the scroll thumb by the specified amount in pixels (negative number to scroll to previous, positive to scroll next). Similar to `addScrollTop` but works for both vertical and horizontal virtualization.


- `IgxGrid`, `IgxTreeGrid`, `IgxHierarchicalGrid`, `IgxPivotGrid`
Copy link
Member

Choose a reason for hiding this comment

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

All of those events are not really available for the IgxPivotGrid

damyanpetev
damyanpetev previously approved these changes Feb 5, 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.

Would still love to try to get a migration for this before the official release (though it'll likely be somewhat challenging) so we can guide users to drop the deprecated properties on the new event interfaces for rowAdd/Delete.
LGTM otherwise.

@hanastasov hanastasov merged commit f8bf9a9 into master Feb 5, 2024
5 of 6 checks passed
@hanastasov hanastasov deleted the ddincheva/deprecatedPropRemovalMaster-16.1 branch February 5, 2024 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grid version: 17.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.

Deprecate rowID and rowData in interfaces
7 participants