Skip to content

feat(ToolResponse): Make card optional#680

Merged
rebeccaalpert merged 1 commit intopatternfly:mainfrom
rebeccaalpert:optional-card-tool-response
Sep 15, 2025
Merged

feat(ToolResponse): Make card optional#680
rebeccaalpert merged 1 commit intopatternfly:mainfrom
rebeccaalpert:optional-card-tool-response

Conversation

@rebeccaalpert
Copy link
Member

@rebeccaalpert rebeccaalpert commented Sep 15, 2025

Ask from Shadowbot team to make the tool response card optional - easy for us to do and it's non-breaking.

@rebeccaalpert rebeccaalpert linked an issue Sep 15, 2025 that may be closed by this pull request
@patternfly-build
Copy link

patternfly-build commented Sep 15, 2025

Copy link
Collaborator

@thatblindgeye thatblindgeye left a comment

Choose a reason for hiding this comment

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

This lgtm, just one question below, but otherwise ✅

<Divider {...toolResponseCardDividerProps} />
<CardBody {...toolResponseCardBodyProps}>{cardBody}</CardBody>
</Card>
{cardTitle && cardBody && (
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would we want to extend the ask a bit and allow rendering only title or body, rather than all-or-nothing?

Copy link
Member Author

Choose a reason for hiding this comment

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

All set if you want to take a look!

@rebeccaalpert rebeccaalpert force-pushed the optional-card-tool-response branch from 20df808 to 7d9e723 Compare September 15, 2025 18:49
@rebeccaalpert rebeccaalpert merged commit 4c75821 into patternfly:main Sep 15, 2025
7 checks passed
@github-actions
Copy link

🎉 This PR is included in version 6.4.0-prerelease.23 🎉

The release is available on:

Your semantic-release bot 📦🚀

rebeccaalpert added a commit to rebeccaalpert/virtual-assistant that referenced this pull request Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make card optional in tool response

3 participants