Skip to content

fix: Adding new follow up props to Frontend tools#3589

Open
ChloeDumit wants to merge 2 commits intomainfrom
one-follow-up
Open

fix: Adding new follow up props to Frontend tools#3589
ChloeDumit wants to merge 2 commits intomainfrom
one-follow-up

Conversation

@ChloeDumit
Copy link
Contributor

Description

We need to avoid having follow up after any upselling frontend tool, adding prop to avoid that

@ChloeDumit ChloeDumit requested a review from a team as a code owner March 4, 2026 11:30
Copilot AI review requested due to automatic review settings March 4, 2026 11:30
@github-actions github-actions bot added the react Changes affect packages/react label Mar 4, 2026
@ChloeDumit ChloeDumit changed the title Fix: Adding new follow up props to Frontend tools fix: Adding new follow up props to Frontend tools Mar 4, 2026
@github-actions github-actions bot added the fix label Mar 4, 2026
Copy link
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 adds followUp: false to all five UpsellingKit frontend CopilotKit action registrations to prevent the AI from generating follow-up messages after upselling components are displayed. It also changes their available value from "disabled" to "frontend" to enable the actions to be called from the frontend. Additionally, a story args block is added to the F0OneSwitch Default story.

Changes:

  • All five use*Action hooks in UpsellingKit/ai are updated: available changed from "disabled" to "frontend" and followUp: false added.
  • F0OneSwitch Default story gains an args block (with an onToggle handler).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
F0OneSwitch.stories.tsx Adds a default args block to the Default story — but uses a non-existent onToggle prop
useQuestionCardAction.tsx Sets available: "frontend" and followUp: false
useModuleCardAction.tsx Sets available: "frontend" and followUp: false
useFAQCardAction.tsx Sets available: "frontend" and followUp: false
useDemoCardAction.tsx Sets available: "frontend" and followUp: false
useBookAMeetingCardAction.tsx Sets available: "frontend" and followUp: false

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

✅ No New Circular Dependencies

No new circular dependencies detected. Current count: 0

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

📦 Alpha Package Version Published

Use pnpm i github:factorialco/f0#npm/alpha-pr-3589 to install the package

Use pnpm i github:factorialco/f0#012801948058ce6e7fec0464731cf88074609ac0 to install this specific commit

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

🔍 Visual review for your branch is published 🔍

Here are the links to:

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

Coverage Report for packages/react

Status Category Percentage Covered / Total
🔵 Lines 41.94% 8513 / 20298
🔵 Statements 41.29% 8738 / 21158
🔵 Functions 32.92% 1862 / 5656
🔵 Branches 31.63% 5052 / 15972
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/react/src/sds/UpsellingKit/ai/F0BookAMeetingCard/useBookAMeetingCardAction.tsx 25% 0% 0% 25% 11-53
packages/react/src/sds/UpsellingKit/ai/F0DemoCard/useDemoCardAction.tsx 26.66% 0% 0% 28.57% 10, 13-18, 26-91
packages/react/src/sds/UpsellingKit/ai/F0FAQCard/useFAQCardAction.tsx 25% 0% 0% 25% 11-73
packages/react/src/sds/UpsellingKit/ai/F0ModuleCard/useModuleCardAction.tsx 25% 0% 0% 25% 11-53
packages/react/src/sds/UpsellingKit/ai/F0QuestionCard/useQuestionCardAction.tsx 7.14% 0% 0% 7.69% 18-26, 68-189
Generated in workflow #11483 for commit 79e1f00 by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix react Changes affect packages/react

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants