Skip to content

chore: Salesforce - Add additional logging#18959

Merged
zomars merged 5 commits intomainfrom
saleforce-more-logging
Jan 28, 2025
Merged

chore: Salesforce - Add additional logging#18959
zomars merged 5 commits intomainfrom
saleforce-more-logging

Conversation

@joeauyeung
Copy link
Copy Markdown
Contributor

@joeauyeung joeauyeung commented Jan 28, 2025 •
edited by zomars
Loading

What does this PR do?

  • Fixes #XXXX (GitHub issue number)
  • Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description)

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

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my changes generate no new warnings

@graphite-app graphite-app Bot requested a review from a team January 28, 2025 17:56
@keithwillcode keithwillcode added consumer core area: core, team members only labels Jan 28, 2025
@dosubot dosubot Bot added the crm-apps area: crm apps, salesforce, hubspot, close.com, sendgrid label Jan 28, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Jan 28, 2025

Graphite Automations

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

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 28, 2025 •
edited
Loading

E2E results are ready!

keithwillcode
keithwillcode reviewed Jan 28, 2025
View reviewed changes
Comment thread packages/app-store/salesforce/lib/CrmService.ts
@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 28, 2025 •
edited
Loading

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

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 28, 2025 9:33pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 28, 2025 9:33pm

@zomars zomars merged commit 2a5e93b into main Jan 28, 2025
@zomars zomars deleted the saleforce-more-logging branch January 28, 2025 22:08
zomars added a commit that referenced this pull request Jan 28, 2025
@joeauyeung @zomars
chore: Salesforce - Add additional logging (#18959)
c97c513 
Co-authored-by: Omar López <zomars@me.com>
MuhammadAimanSulaiman pushed a commit to hit-pay/cal.com that referenced this pull request Feb 25, 2025
@joeauyeung @zomars @MuhammadAimanSulaiman
chore: Salesforce - Add additional logging (calcom#18959)
8a4c8f8 
Co-authored-by: Omar López <zomars@me.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@keithwillcode keithwillcode keithwillcode left review comments

@hbjORbj hbjORbj hbjORbj approved these changes

+1 more reviewer

@zomars zomars zomars approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

consumer core area: core, team members only crm-apps area: crm apps, salesforce, hubspot, close.com, sendgrid ready-for-e2e

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@joeauyeung @keithwillcode @zomars @hbjORbj