-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Add the email property to the create customer component of quickbooks #16724
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
base: master
Are you sure you want to change the base?
Add the email property to the create customer component of quickbooks #16724
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
Someone is attempting to deploy a commit to the Pipedreamers Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughA new optional property, Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CreateCustomerAction
participant QuickBooksAPI
User->>CreateCustomerAction: Provide customer details (optionally with primaryEmailAddr)
CreateCustomerAction->>QuickBooksAPI: Send create customer payload (includes PrimaryEmailAddr if provided)
QuickBooksAPI-->>CreateCustomerAction: Return customer creation result
CreateCustomerAction-->>User: Return result
Poem
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
components/quickbooks/quickbooks.app.mjsOops! Something went wrong! :( ESLint: 8.57.1 Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'jsonc-eslint-parser' imported from /eslint.config.mjs components/quickbooks/actions/create-customer/create-customer.mjsOops! Something went wrong! :( ESLint: 8.57.1 Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'jsonc-eslint-parser' imported from /eslint.config.mjs Note ⚡️ AI Code Reviews for VS Code, Cursor, WindsurfCodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback. Note ⚡️ Faster reviews with cachingCodeRabbit now supports caching for code and dependencies, helping speed up reviews. This means quicker feedback, reduced wait times, and a smoother review experience overall. Cached data is encrypted and stored securely. This feature will be automatically enabled for all accounts on May 16th. To opt out, configure ✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Thank you so much for submitting this! We've added it to our backlog to review, and our team has been notified. |
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/quickbooks/actions/create-customer/create-customer.mjs (1)
72-72
: Consider conditionally including the PrimaryEmailAddr field.The implementation works correctly to pass the email to the QuickBooks API. However, you could consider only including this field when a value is provided.
- PrimaryEmailAddr: {Address: this.primaryEmailAddr}, + ...(this.primaryEmailAddr ? { PrimaryEmailAddr: {Address: this.primaryEmailAddr} } : {}),This would prevent sending empty objects when no email is provided.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting
📒 Files selected for processing (2)
components/quickbooks/actions/create-customer/create-customer.mjs
(3 hunks)components/quickbooks/quickbooks.app.mjs
(1 hunks)
🔇 Additional comments (3)
components/quickbooks/quickbooks.app.mjs (1)
424-429
: LGTM: The new primaryEmailAddr property is well-defined.The property definition is consistent with other properties in the file, using the appropriate type and making it optional. The label and description are clear and concise.
components/quickbooks/actions/create-customer/create-customer.mjs (2)
8-8
: Version increment is appropriate.Version has been incremented from 0.1.8 to 0.1.9, which is suitable for this feature addition.
48-53
: LGTM: The primaryEmailAddr prop is properly referenced.The property is correctly referenced from the QuickBooks app propDefinitions.
WHY
The current implementation of the QuickBooks "Create Customer" action lacks a feature: the ability to set a customer's email address during creation.
The QuickBooks API fully supports setting email addresses during customer creation, so this change simply exposes existing API capabilities that were previously unavailable in the Pipedream component.
Summary by CodeRabbit