Skip to content

Indicate that component filter args must always be strings#71

Merged
derek-etherton-opslevel merged 2 commits intomainfrom
fix-int-filter-args
Jul 28, 2025
Merged

Indicate that component filter args must always be strings#71
derek-etherton-opslevel merged 2 commits intomainfrom
fix-int-filter-args

Conversation

@derek-etherton-opslevel
Copy link
Copy Markdown
Contributor

Problem

Our components API only accepts strings for args, but agents will often try to pass numbers.

Solution

Add a note to the input that Arg must always be string.

Checklist

  • I have run this code, and it appears to resolve the stated issue.
  • This PR has no user interface changes or has already received approval from product management to change the interface.
  • Make a changie entry that explains the customer facing outcome of this change.

@derek-etherton-opslevel derek-etherton-opslevel force-pushed the fix-int-filter-args branch 3 times, most recently from 56f265c to d4a80c9 Compare July 28, 2025 18:45
Copy link
Copy Markdown
Contributor

@rocktavious rocktavious left a comment

Choose a reason for hiding this comment

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

Look at that it even fixes the nesting!

@derek-etherton-opslevel derek-etherton-opslevel merged commit d6a63fc into main Jul 28, 2025
2 checks passed
@derek-etherton-opslevel derek-etherton-opslevel deleted the fix-int-filter-args branch July 28, 2025 18:49
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.

2 participants