Skip to content

fix(ENG-12152): render description and tooltips for falcon connectors#118

Merged
adefreitas merged 1 commit intoStackOneHQ:mainfrom
adefreitas:ENG-12152/render-description-and-tooltips-for-falcon-connectors
Mar 18, 2026
Merged

fix(ENG-12152): render description and tooltips for falcon connectors#118
adefreitas merged 1 commit intoStackOneHQ:mainfrom
adefreitas:ENG-12152/render-description-and-tooltips-for-falcon-connectors

Conversation

@adefreitas
Copy link
Copy Markdown
Collaborator

@adefreitas adefreitas commented Mar 18, 2026

Before

Screenshot 2026-03-18 at 15 37 06

After

Screenshot 2026-03-18 at 15 35 39

Summary by cubic

Fixes missing tooltips and descriptions for Falcon connectors in the Auth Hub by falling back to field.description and field.tooltip when guide is absent. Restores parity with V2 connectors and improves account linking.

  • Bug Fixes
    • Render tooltips/descriptions for Falcon fields by using guide values when present, otherwise field.description/field.tooltip.
    • Add optional description and tooltip to ConnectorConfigField; addresses ENG-12152.

Written for commit 213a0b2. Summary will update on new commits.

Copilot AI review requested due to automatic review settings March 18, 2026 15:37
@adefreitas adefreitas enabled auto-merge (squash) March 18, 2026 15:37
@adefreitas adefreitas changed the title fix: render description and tooltips for falcon connectors fix(ENG-12152): render description and tooltips for falcon connectors Mar 18, 2026
@adefreitas adefreitas disabled auto-merge March 18, 2026 15:38
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic analysis

No issues found across 2 files

Linked issue analysis

Linked issue: ENG-12152: Fix tooltip and description display for Falcon connectors in the hub

Status Acceptance criteria Notes
Tooltips are displayed for Falcon connectors Added tooltip fallback to field.tooltip in IntegrationFields.tsx
Descriptions are displayed for Falcon connectors Added description fallback to field.description in IntegrationFields.tsx
Behavior is consistent with V2 connectors (guide.* still preferred) Code prefers guide.* then falls back to field.*
ConnectorConfigField type includes description and tooltip Added optional description and tooltip in types.ts
Apply tooltip/description fallback across all field renderers (secret, input, textarea, select) All render paths updated in IntegrationFields.tsx
⚠️ Fix aids successful account linking (UX/outcome) UI help text restored but no metrics or tests proving improved linking

Copy link
Copy Markdown

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 updates the integration picker field rendering to display per-field descriptions and tooltips for Falcon connector configuration fields (in addition to the existing legacy “guide” metadata), aligning the UI with the expected connector config shape.

Changes:

  • Extend ConnectorConfigField to support optional description and tooltip.
  • Update field rendering to fall back from field.guide to the new top-level field.description / field.tooltip across input, textarea, and dropdown fields.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/modules/integration-picker/types.ts Adds optional description/tooltip to the config field type so Falcon configs can supply field help text directly.
src/modules/integration-picker/components/IntegrationFields.tsx Uses field.guide?.* ?? field.* so the UI renders help text for both legacy and Falcon connector field shapes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@adefreitas adefreitas merged commit 6777742 into StackOneHQ:main Mar 18, 2026
9 checks passed
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