Skip to content

47998 frontend [ templates ] Show Draft for inactive templates without integrations#260

Open
aicarma-artyom-maslov wants to merge 3 commits into
masterfrom
frontend/templates/47998__fix_inactive_template_card
Open

47998 frontend [ templates ] Show Draft for inactive templates without integrations#260
aicarma-artyom-maslov wants to merge 3 commits into
masterfrom
frontend/templates/47998__fix_inactive_template_card

Conversation

@aicarma-artyom-maslov

@aicarma-artyom-maslov aicarma-artyom-maslov commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Note

Low Risk
Templates list UI and display rules only; template edit draft warnings are untouched.

Overview
Inactive template cards no longer treat account-wide webhooks or public visibility as reasons to show integration badges or the draft warning tooltip. The footer now uses hasTemplateCardIntegrations (webhooks excluded via TEMPLATE_CARD_INTEGRATIONS_EXCLUDE) and only shows the integrations strip when the template is active or has other connected integrations.

For inactive templates with no such integrations, users see a plain Draft label—no integrated/not-integrated chip and no warning icon. isPublic is dropped from the card footer props and wiring; card-related types move into TemplatesUser/types.ts.

checkShowDraftTemplateWarning is unchanged in behavior (still considers public + any integrations) and remains for template edit controls; this PR adds unit tests for it alongside the new helper.

Reviewed by Cursor Bugbot for commit 240ff07. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Show Draft label only for inactive templates without non-webhook integrations

  • Changes the draft warning logic in TemplateCardFooter so it only shows for inactive templates that have no non-webhook integrations, replacing the previous isPublic-based check.
  • Introduces hasTemplateCardIntegrations in templateIntegrations.ts and TEMPLATE_CARD_INTEGRATIONS_EXCLUDE (set to [EIntegrations.Webhooks]) to drive this logic.
  • The integrations indicator is hidden for inactive templates that have no non-webhook integrations.
  • Removes isPublic prop from TemplateCard and TemplateCardFooter, consolidating prop types into types.ts.
  • Behavioral Change: templates that were previously inactive and public would show a draft warning; now only inactive templates with non-webhook integrations show the integrations indicator instead of the draft label.

Macroscope summarized 240ff07.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e7f4059. Configure here.

Comment thread frontend/src/public/components/Templates/TemplatesUser/TemplateCardFooter.tsx Outdated
@aicarma-artyom-maslov aicarma-artyom-maslov changed the title 47998 fix(templates): show Draft for inactive templates without integrations 47998 frontend [ templates ] Show Draft for inactive templates without integrations Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend Web client changes request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant