Skip to content

fix: add missing resourceDrillInOnLoad prop to ExpressionContainer#1746

Merged
paustint merged 1 commit into
mainfrom
bug/1743
May 27, 2026
Merged

fix: add missing resourceDrillInOnLoad prop to ExpressionContainer#1746
paustint merged 1 commit into
mainfrom
bug/1743

Conversation

@paustint
Copy link
Copy Markdown
Contributor

Related fields in filter group did not trigger fetching data from cache/sfdc

fixes #1743

Related fields in filter group did not trigger fetching data from cache/sfdc

fixes #1743
Copilot AI review requested due to automatic review settings May 27, 2026 00:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes drill-in behavior for grouped filter rows by ensuring the resourceDrillInOnLoad callback is forwarded to ExpressionConditionRow instances rendered inside an ExpressionGroup, enabling related-field selection to correctly trigger resource loading (cache/SFDC) as expected.

Changes:

  • Pass resourceDrillInOnLoad to grouped (child) ExpressionConditionRow components.
  • Align behavior between top-level condition rows and group condition rows for resource drill-in on initial load.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@paustint paustint merged commit c124f79 into main May 27, 2026
9 checks passed
@paustint paustint deleted the bug/1743 branch May 27, 2026 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When using Group Filter, Cannot select cross object field

2 participants