Skip to content

improvement(search): simplify integration setup and connection actions - #7761

Merged
waleedlatif1 merged 7 commits into
stagingfrom
codex/integration-actions-simplify
Sep 11, 2026
Merged

improvement(search): simplify integration setup and connection actions#7761
waleedlatif1 merged 7 commits into
stagingfrom
codex/integration-actions-simplify

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Show one member row per integration, reuse authorized accounts across sources, and keep new-account actions inside account dropdowns.
  • Consolidate sync actions with confirmation and cooldowns; preserve source permissions when removing connections.
  • Align setup controls with chip components, simplify Slack and GitLab forms, restore selector arrows, and refresh Search guides and screenshots.

Type of Change

  • Bug fix
  • Documentation
  • Other: UI improvements

Testing

  • Full application suite: 49,914 passed and 185 skipped with four workers. Shared components: 222 passed. Focused setup/action regression tests: 156 passed.
  • App and component TypeScript, lint, all 46 repository audits, and docs manifest checks passed. All 10 Search guides compile and their 25 image references resolve.
  • Reviewed current setup and action states with live local UI and component fixtures; recaptured changed styling. External provider authorization and indexing were not repeated for every provider in this pass.
  • The default fully parallel local run hit one timeout in an unchanged metadata-resolution test. Its 52-test file and the complete application suite passed with lower worker concurrency, without changing timeouts or assertions. All CI test shards, build, lint, and database checks passed. No local production build.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 11, 2026 3:26am UTC

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@greptile-apps

greptile-apps Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the previously reported pagination problem is resolved and no actionable new failures remain.

Summary

  • Consolidates member integrations into one provider row and reuses authorized accounts across configured sources.
  • Adds explicit bounded pagination for checking additional connections without falsely reporting a partial inventory as connected.
  • Consolidates connector sync, pause, resume, resync, removal, and cooldown behavior.
  • Updates shared chip controls, provider setup forms, tests, Search documentation, and screenshots.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Load integration row] --> B{Loaded source needs authorization?}
  B -->|Yes| C[Show Connect or Reconnect]
  B -->|No| D{More source pages exist?}
  D -->|Yes| E[Show Check connections]
  E --> F[Load one bounded page]
  F --> B
  D -->|No| G{Central integration?}
  G -->|Yes| H[Connected by organization]
  G -->|No| I[Show final member connection state]
Loading

Reviews (2) · Last reviewed commit: "fix(search): expose incomplete connectio..."

Comment thread apps/sim/app/o/[organizationId]/integrations/member-integration-row.tsx Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1
waleedlatif1 merged commit 1be4328 into staging Sep 11, 2026
33 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/integration-actions-simplify branch September 11, 2026 03:31
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.

1 participant