- 
                Notifications
    You must be signed in to change notification settings 
- Fork 5.5k
Revert "Revert "Webflow migration follow-up adjustments"" #14826
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -1,10 +1,10 @@ | ||
| import app from "../../webflow_v2.app.mjs"; | ||
| import app from "../../webflow.app.mjs"; | ||
|  | ||
| export default { | ||
| key: "webflow_v2-get-collection", | ||
| key: "webflow-get-collection", | ||
| There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Component metadata needs attention for complete consistency The verification revealed two inconsistencies that need to be addressed: 
 🔗 Analysis chainComponent metadata updates look consistent The changes align with the Webflow migration follow-up: 
 Let's verify the consistency of these changes across other Webflow components: Also applies to: 6-7 Let me verify the component keys to ensure complete consistency. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify consistent metadata updates across Webflow components
# Expected: All components should follow the same pattern
echo "Checking for any remaining versioned keys..."
rg -g '*.{mjs,js}' 'key: "webflow_v2-' components/webflow/
echo "Checking version numbers..."
rg -g '*.{mjs,js}' 'version: "' components/webflow/
Length of output: 2560 Script: #!/bin/bash
# Check all Webflow component keys
rg -g '*.{mjs,js}' 'key: "webflow-' components/webflow/
Length of output: 2765 | ||
| name: "Get Collection", | ||
| description: "Get a collection. [See the docs here](https://developers.webflow.com/#get-collection-with-full-schema)", | ||
| version: "0.0.1", | ||
| description: "Get a collection. [See the documentation](https://developers.webflow.com/data/reference/cms/collections/get)", | ||
| version: "1.0.0", | ||
| type: "action", | ||
| props: { | ||
| app, | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -1,10 +1,10 @@ | ||
| import app from "../../webflow_v2.app.mjs"; | ||
| import app from "../../webflow.app.mjs"; | ||
|  | ||
| export default { | ||
| key: "webflow_v2-get-order", | ||
| key: "webflow-get-order", | ||
| name: "Get Order", | ||
| description: "Get info on an order. [See the docs here](https://developers.webflow.com/#get-order)", | ||
| version: "0.0.1", | ||
| description: "Get info on an order. [See the documentation](https://developers.webflow.com/data/reference/ecommerce/orders/get)", | ||
| version: "1.0.0", | ||
| There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Version 1.0.0 is consistent with most actions, but one action needs attention Almost all Webflow actions are at version 1.0.0, but the  
 🔗 Analysis chainVersion bump to 1.0.0 is appropriate The version increment to 1.0.0 aligns with the broader component updates. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify consistent version numbers across components
# Check version numbers across Webflow components
rg '"version":.*"[0-9]+\.[0-9]+\.[0-9]+"' components/webflow/
Length of output: 117 Script: #!/bin/bash
# Let's check version numbers in all action files within webflow components
fd -e mjs -e js . components/webflow/actions -x grep -l "version.*[0-9]"
# Also get the content of these files to verify version consistency
fd -e mjs -e js . components/webflow/actions -x grep -A 1 -B 1 "version.*[0-9]"
Length of output: 4481 | ||
| type: "action", | ||
| props: { | ||
| app, | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -1,10 +1,10 @@ | ||
| import app from "../../webflow_v2.app.mjs"; | ||
| import app from "../../webflow.app.mjs"; | ||
|  | ||
| export default { | ||
| key: "webflow_v2-list-collection-items", | ||
| key: "webflow-list-collection-items", | ||
| name: "List Collection Items", | ||
| description: "List Items of a collection. [See the docs here](https://developers.webflow.com/#get-all-items-for-a-collection)", | ||
| version: "0.0.1", | ||
| description: "List Items of a collection. [See the documentation](https://developers.webflow.com/data/reference/cms/collection-items/bulk-items/list-items)", | ||
| version: "1.0.0", | ||
| 
      Comment on lines
    
      +4
     to 
      +7
    
   There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Version inconsistency found in Webflow components The metadata changes are mostly consistent across components, but there's one inconsistency: 
 🔗 Analysis chainVerify version bump consistency across components. The metadata changes look good: 
 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify consistent versioning and key naming across Webflow components
# Check for any remaining v2 keys
rg '"webflow_v2-' components/webflow/
# Check for version consistency across components
rg '"version":\s*"[^"]*"' components/webflow/ --type js
Length of output: 97 Script: #!/bin/bash
# Let's try a different approach to check component metadata
# Search for key and version fields in action files
rg -A 2 '"key":\s*"[^"]*"' --type js components/webflow/actions/
# Also check for any remaining v2 references with a broader pattern
rg -i 'webflow.*v2' components/webflow/
Length of output: 108 Script: #!/bin/bash
# Try a different approach using the .mjs extension and looking at the actual content structure
rg -A 3 'key:.*"webflow' components/webflow/actions/**/*.mjs
# Also check version fields specifically in .mjs files
rg 'version:.*"[0-9]+\.[0-9]+\.[0-9]+"' components/webflow/actions/**/*.mjs
Length of output: 12571 | ||
| type: "action", | ||
| props: { | ||
| app, | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -1,10 +1,10 @@ | ||
| import app from "../../webflow_v2.app.mjs"; | ||
| import app from "../../webflow.app.mjs"; | ||
|  | ||
| export default { | ||
| key: "webflow_v2-list-collections", | ||
| key: "webflow-list-collections", | ||
| name: "List Collections", | ||
| description: "List collections. [See the docs here](https://developers.webflow.com/#list-collections)", | ||
| version: "0.0.1", | ||
| description: "List collections. [See the documentation](https://developers.webflow.com/data/reference/cms/collections/list)", | ||
| version: "1.0.0", | ||
| There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Version inconsistency detected in Webflow components All Webflow components are at version 1.0.0 except for  
 🔗 Analysis chainVersion bump is appropriate The version bump to 1.0.0 correctly reflects the breaking changes in the component key and import path. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check for consistent version bumps across related Webflow components
# Find all Webflow component files and check their versions
echo "Checking versions in Webflow components..."
fd -e mjs . components/webflow/ -x grep -l "version:" {} | xargs grep "version:"
Length of output: 2457 | ||
| type: "action", | ||
| props: { | ||
| app, | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -1,10 +1,10 @@ | ||
| import app from "../../webflow_v2.app.mjs"; | ||
| import app from "../../webflow.app.mjs"; | ||
|  | ||
| export default { | ||
| key: "webflow_v2-list-sites", | ||
| key: "webflow-list-sites", | ||
| name: "List Sites", | ||
| description: "List sites. [See the docs here](https://developers.webflow.com/#list-sites)", | ||
| version: "0.0.1", | ||
| description: "List sites. [See the documentation](https://developers.webflow.com/data/reference/sites/list)", | ||
| version: "1.0.0", | ||
| 
      Comment on lines
    
      +4
     to 
      +7
    
   There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Update version number for update-item-inventory action The verification reveals that most changes are consistent across Webflow actions: 
 However, the  •  🔗 Analysis chainVerify consistency of version and key changes across Webflow actions The changes include: 
 These changes suggest a breaking change and standardization effort. Let me gather more information to verify the key format and version numbers. Let me try one more search with a different pattern to catch the key and version fields. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify consistency of version and key changes across Webflow actions
# Check for any remaining v2 references in keys
echo "=== Checking for remaining v2 references in keys ==="
rg '"webflow_v2-' components/webflow/
# Check version numbers across Webflow actions
echo "=== Checking version numbers ==="
rg '"version":' components/webflow/actions/
# Verify documentation link format
echo "=== Checking documentation link format ==="
rg 'developers\.webflow\.com/data/reference' components/webflow/
Length of output: 6077 Script: #!/bin/bash
# Check key format and version numbers in all Webflow action files
# Check all keys in action files
echo "=== Checking action keys format ==="
rg '"key":' components/webflow/actions/
# Check version numbers in action files
echo "=== Checking version numbers in actions ==="
rg '"version":' components/webflow/actions/
Length of output: 265 Script: #!/bin/bash
# Search for key and version fields with more flexible pattern
echo "=== Checking action keys format ==="
rg "key: " components/webflow/actions/
echo "=== Checking version numbers ==="
rg "version: " components/webflow/actions/
Length of output: 3516 | ||
| type: "action", | ||
| props: { | ||
| app, | ||
|  | ||
| Original file line number | Diff line number | Diff line change | 
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| { | ||
| "name": "@pipedream/webflow", | ||
| "version": "0.4.6", | ||
| "version": "1.0.0", | ||
| "description": "Pipedream Webflow Components", | ||
| "main": "webflow.app.mjs", | ||
| "keywords": [ | ||
|  | @@ -10,10 +10,9 @@ | |
| "homepage": "https://pipedream.com/apps/webflow", | ||
| "author": "Pipedream <[email protected]> (https://pipedream.com/)", | ||
| "dependencies": { | ||
| "@pipedream/platform": "^1.1.0", | ||
| "webflow-api": "1.3.1" | ||
| "@pipedream/platform": "^3.0.3", | ||
| "webflow-api": "2.4.2" | ||
| }, | ||
| "gitHead": "e12480b94cc03bed4808ebc6b13e7fdb3a1ba535", | ||
| "publishConfig": { | ||
| "access": "public" | ||
| } | ||
|  | ||
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.
💡 Codebase verification
Version inconsistency found in Webflow components
Almost all Webflow components are at version 1.0.0, except for:
webflow-update-item-inventorywhich is at version 0.0.5This inconsistency should be addressed to maintain version alignment across the Webflow integration components.
🔗 Analysis chain
Verify the impact of version bump to 1.0.0
The metadata changes look good:
Let's verify version consistency across Webflow components:
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 57
Script:
Length of output: 70
Script:
Length of output: 9076