Skip to content

Conversation

@ViaSocket-Git
Copy link
Collaborator

  • Moved TriggersList from SetupView to InputSection for better component organization
  • Added "Triggers" radio button option to BridgeTypeToggle for non-embed users
  • Updated TriggersList styling with improved spacing, max-width constraints, and hover effects
  • Replaced InfoTooltip label with CircleQuestionMark icon for cleaner trigger label design
  • Enhanced trigger card layout with better padding and background color transitions

- Moved TriggersList from SetupView to InputSection for better component organization
- Added "Triggers" radio button option to BridgeTypeToggle for non-embed users
- Updated TriggersList styling with improved spacing, max-width constraints, and hover effects
- Replaced InfoTooltip label with CircleQuestionMark icon for cleaner trigger label design
- Enhanced trigger card layout with better padding and background color transitions
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +29 to +32
{!isEmbedUser && bridgeType === 'trigger' &&
<div className="w-full pt-4 cursor-default flex flex-wrap justify-between items-start gap-2">
<div className="flex-1 min-w-[220px] max-w-md">
<TriggersList params={params} />

Choose a reason for hiding this comment

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

P1 Badge Restore trigger management for image bridges

Moving TriggersList into InputSection now only renders triggers when NonImageModelConfig is used. SetupView routes image models through the image branch that never renders InputSection, so selecting bridge type trigger on an image bridge no longer shows any trigger list or add button even though the trigger option is still available. This hides trigger management for all image bridges with bridgeType set to trigger.

Useful? React with 👍 / 👎.

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