Skip to content

fix: Salesforce bugs#19250

Merged
Udit-takkar merged 4 commits intomainfrom
salesforce-fixes
Feb 12, 2025
Merged

fix: Salesforce bugs#19250
Udit-takkar merged 4 commits intomainfrom
salesforce-fixes

Conversation

@joeauyeung
Copy link
Copy Markdown
Contributor

@joeauyeung joeauyeung commented Feb 11, 2025

What does this PR do?

  • When writing to a record when a booking is incomplete, the lead query would overwrite the contact query
  • When creating contacts under an account, we were not searching if contacts were created

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

@graphite-app graphite-app Bot requested a review from a team February 11, 2025 19:49
@keithwillcode keithwillcode added consumer core area: core, team members only labels Feb 11, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Feb 11, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (02/11/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (02/11/25)

1 label was added to this PR based on Keith Williams's automation.

@dosubot dosubot Bot added crm-apps area: crm apps, salesforce, hubspot, close.com, sendgrid 🐛 bug Something isn't working labels Feb 11, 2025
);

if (contactsQuery.records.length) {
if (contactsQuery.records.length > 0) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of an if statement this is the ideal place to extract into a sub function :)

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cal-com-ui-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 12, 2025 8:19am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Feb 12, 2025 8:19am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Feb 12, 2025 8:19am

Copy link
Copy Markdown
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean! 🚀 well done fixup

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 11, 2025

E2E results are ready!

@Udit-takkar Udit-takkar enabled auto-merge (squash) February 12, 2025 08:06
@Udit-takkar Udit-takkar merged commit 3fe83bb into main Feb 12, 2025
@Udit-takkar Udit-takkar deleted the salesforce-fixes branch February 12, 2025 08:24
MuhammadAimanSulaiman pushed a commit to hit-pay/cal.com that referenced this pull request Feb 25, 2025
* incompleteBooking only search for leads if contact is not found

* Fix logic

* Abstract to functions

---------

Co-authored-by: Udit Takkar <53316345+Udit-takkar@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working consumer core area: core, team members only crm-apps area: crm apps, salesforce, hubspot, close.com, sendgrid ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants