-
Notifications
You must be signed in to change notification settings - Fork 2.6k
feat(webflow): added collection, item, & site selectors for webflow #2368
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile OverviewGreptile SummaryThis PR adds comprehensive UI selectors for Webflow integration, enabling users to select sites, collections, and items through dropdown menus instead of manually entering IDs. Key Changes:
Architecture: Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant User
participant Selector as UI Selectors
participant Routes as API Routes
participant Webflow
User->>Selector: Pick credential
Selector->>Routes: Fetch sites
Routes->>Webflow: List sites
Webflow-->>Routes: Sites data
Routes-->>Selector: Formatted sites
User->>Selector: Pick site
Selector->>Routes: Fetch collections
Routes->>Webflow: List collections for site
Webflow-->>Routes: Collections data
Routes-->>Selector: Formatted collections
User->>Selector: Pick collection
Selector->>Routes: Fetch items
Routes->>Webflow: List items for collection
Webflow-->>Routes: Items data
Routes-->>Selector: Formatted items
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additional Comments (1)
-
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/project-selector/project-selector-input.tsx, line 46-53 (link)style: Dead code:
isLinear,linearCredential, andjiraCredentialare no longer used after the credential resolution simplification.
10 files reviewed, 1 comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
17 files reviewed, 1 comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
17 files reviewed, no comments
Summary
Type of Change
Testing
Tested manually
Checklist