Skip to content

fix(elements): skip event detail generation for filtering expression tree - master#17371

Merged
dkamburov merged 1 commit into
masterfrom
iminchev/fix-filtering-tree-details
Jul 23, 2026
Merged

fix(elements): skip event detail generation for filtering expression tree - master#17371
dkamburov merged 1 commit into
masterfrom
iminchev/fix-filtering-tree-details

Conversation

@IMinchev64

Copy link
Copy Markdown
Contributor

Related to #IgniteUI/igniteui-blazor#230

Description

Motivation / Context

Type of Change (check all that apply):

  • Bug fix
  • New functionality
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation
  • Demos
  • CI/CD
  • Tests
  • Changelog
  • Skills/Agents

Component(s) / Area(s) Affected:

How Has This Been Tested?

  • Unit tests
  • Manual testing
  • Automated e2e tests

Test Configuration:

  • Angular version:
  • Browser(s):
  • OS:

Screenshots / Recordings

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
  • Accessibility (ARIA, keyboard navigation, focus management) has been verified

Copilot AI review requested due to automatic review settings July 1, 2026 13:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds metadata annotations to the filtering expression tree types so the Angular Elements event pipeline can skip generating event.detail payloads for this structure (aligning with the reported issue context).

Changes:

  • Marked IFilteringExpressionsTree with /* skipEventDetails */.
  • Marked FilteringExpressionsTree with /* skipEventDetails */.

@dkamburov
dkamburov requested a review from skrustev July 17, 2026 11:58
@dkamburov dkamburov added ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification labels Jul 23, 2026
@dkamburov
dkamburov merged commit 9726bdf into master Jul 23, 2026
7 checks passed
@dkamburov
dkamburov deleted the iminchev/fix-filtering-tree-details branch July 23, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💠 grid: elements version: 22.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.

3 participants