-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Add field in hyros create lead action #18934
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
Add field in hyros create lead action #18934
Conversation
…b.com/PrajwalLokhande2003/pipedream into add-field-in-hyros-create-lead-action
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
Thank you so much for submitting this! We've added it to our backlog to review, and our team has been notified. |
WalkthroughAdded a new optional Changes
Sequence Diagram(s)sequenceDiagram
participant UI as User / Workflow
participant Action as Create Lead Action
participant App as Hyros App (manifest)
participant API as Hyros API
rect rgb(220,235,255)
UI->>Action: invoke action (props + optional stage)
Action->>App: read propDefinition leadStage (if not supplied)
Action->>Action: build data payload {..., stage: stage?}
end
rect rgb(220,255,235)
Action->>API: POST /leads with payload (includes stage when present)
API-->>Action: 200 / lead created
Action-->>UI: return result
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Thanks for submitting this PR! When we review PRs, we follow the Pipedream component guidelines. If you're not familiar, here's a quick checklist:
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
components/hyros/hyros.app.mjs (1)
32-36: Add trailing period for consistency.The description should end with a period to match the style of other propDefinitions in this file (see lines 10, 15, 20, 25, 30).
Apply this diff:
leadStage: { type: "string", label: "Lead Stage", - description: "The name of a stage to be applied to the lead", + description: "The name of a stage to be applied to the lead.", },
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
components/hyros/actions/create-lead/create-lead.mjs(1 hunks)components/hyros/hyros.app.mjs(1 hunks)
🔇 Additional comments (1)
components/hyros/actions/create-lead/create-lead.mjs (1)
58-64: API support for leadStage parameter confirmed.The Hyros Create Lead endpoint supports a lead stage (the "Lead Stage" / stage field), and the implementation correctly includes
leadStagein the API payload through the spread operator.
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
components/hyros/package.json (1)
3-3: Reconsider the version bump strategy.Adding a new optional field is a feature addition, which typically warrants a minor version bump (0.1.0 → 0.2.0), not a patch bump (0.1.0 → 0.1.1). Per semantic versioning, patch bumps are reserved for backward-compatible bug fixes.
If this is intentional (e.g., the field is considered internal), then please clarify the versioning strategy. Otherwise, consider bumping to 0.2.0:
- "version": "0.1.1", + "version": "0.2.0",
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (3)
components/hyros/actions/create-call/create-call.mjs(1 hunks)components/hyros/actions/create-lead/create-lead.mjs(2 hunks)components/hyros/package.json(2 hunks)
✅ Files skipped from review due to trivial changes (1)
- components/hyros/actions/create-call/create-call.mjs
🚧 Files skipped from review as they are similar to previous changes (1)
- components/hyros/actions/create-lead/create-lead.mjs
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2024-12-12T19:23:09.039Z
Learnt from: jcortes
Repo: PipedreamHQ/pipedream PR: 14935
File: components/sailpoint/package.json:15-18
Timestamp: 2024-12-12T19:23:09.039Z
Learning: When developing Pipedream components, do not add built-in Node.js modules like `fs` to `package.json` dependencies, as they are native modules provided by the Node.js runtime.
Applied to files:
components/hyros/package.json
🔇 Additional comments (1)
components/hyros/package.json (1)
16-16: Compatibility with v3.1.0 confirmed; no changes required.The upgrade from
@pipedream/platform@^1.5.1to^3.1.0is safe for this codebase. The implementation uses only stable platform exports:
axios: Low-level HTTP utility wrapper—remains stable across major versionsConfigurationError: Simple exception class—compatible across versionspropDefinitionpattern: Standard SDK feature used with basic types; no advanced typing affected by v3 breaking changesThe code avoids areas with breaking changes in v3 (client initialization, pagination metadata, complex ConfigurableProp typing), making the version bump compatible. Action versions at
0.0.3are appropriate for this migration.
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.
Thank you for your contribution! LGTM! Ready for QA
|
Hi everyone, all test cases are passed! Ready for release! Test reports
|
WHY
Summary by CodeRabbit
New Features
Chores