improvement(search): clarify integration approval and setup - #7673
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Greptile SummaryThis PR replaces the organization integration catalog modal with a persistent provider list, explicit approval switches, and separate setup and management actions.
Confidence Score: 5/5The PR appears safe to merge; no actionable correctness, security, or repository-rule violations were identified. The approval state refreshes through the existing query invalidation path, deactivation preserves configured sources, provider visibility remains aligned with the overview contract, and the documentation asset replacement has no dangling references.
|
| Filename | Overview |
|---|---|
| apps/sim/app/o/[organizationId]/settings/components/integrations/organization-integrations-setup.tsx | Replaces the add-integration modal with a stable catalog, approval switches, explicit setup links, and confirmed deactivation. |
| apps/sim/app/o/[organizationId]/settings/components/integrations/organization-integrations-setup.test.tsx | Adds focused coverage for approval, deactivation, failures, availability, filtering, and access control. |
| apps/docs/content/docs/search/index.mdx | Documents the new allow, setup, management, and deactivation workflow and references the replacement screenshot. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Organization Sources] --> B{Integration approved?}
B -- No --> C[Turn on Allowed in Sim Search]
C --> D[Server confirms approval]
D --> E[Set up integration]
B -- Yes, no sources --> E
B -- Yes, sources exist --> F[Manage sources]
F --> G{Turn approval off?}
G -- Yes --> H[Confirm deactivation]
H --> I[Deactivate while preserving sources and accounts]
G -- No --> F
Reviews (1): Last reviewed commit: "improvement(search): clarify integration..." | Re-trigger Greptile
Summary
Type of Change
Testing
Checklist