Skip to content

fix: Installing Zoho CRM#11161

Merged
kodiakhq[bot] merged 3 commits intomainfrom
fix/install-zoho-crm
Sep 6, 2023
Merged

fix: Installing Zoho CRM#11161
kodiakhq[bot] merged 3 commits intomainfrom
fix/install-zoho-crm

Conversation

@joeauyeung
Copy link
Copy Markdown
Contributor

What does this PR do?

This PR fixes a 400 error when installing Zoho CRM

CleanShot 2023-09-05 at 17 20 10

Fixes # (issue)

Requirement/Documentation

  • If there is a requirement document, please, share it here.
  • If there is ab UI/UX design document, please, share it here.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

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

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 5, 2023

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

Name Status Preview Comments Updated (UTC)
ai ❌ Failed (Inspect) Sep 5, 2023 9:58pm
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2023 9:58pm
cal-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2023 9:58pm
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2023 9:58pm
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2023 9:58pm
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Sep 5, 2023 9:58pm
qa ⬜️ Ignored (Inspect) Visit Preview Sep 5, 2023 9:58pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 2023

Thank you for following the naming conventions! 🙏

@zomars zomars added the core area: core, team members only label Sep 5, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 5, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Sep 5, 2023

No failed tests 🎉

Comment on lines -16 to +25
const redirectUri = WEBAPP_URL + "/api/integrations/zohocrm/callback";
const url = `https://accounts.zoho.com/oauth/v2/auth?scope=ZohoCRM.modules.ALL,ZohoCRM.users.READ,AaaServer.profile.READ&client_id=${client_id}&response_type=code&access_type=offline&redirect_uri=${redirectUri}&state=${state}`;
const params = {
client_id,
response_type: "code",
redirect_uri: WEBAPP_URL + "/api/integrations/zohocrm/callback",
scope: ["ZohoCRM.modules.ALL", "ZohoCRM.users.READ", "AaaServer.profile.READ"],
access_type: "offline",
state,
prompt: "consent",
};
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It was hard to debug what was wrong, so let's separate the URL params. Credit to @murtajaziad in #10429

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think just prompt was missing. Earlier it was working fine, maybe Zoho changed something?

@joeauyeung joeauyeung requested a review from a team September 5, 2023 21:51
@kodiakhq kodiakhq Bot merged commit 868ba43 into main Sep 6, 2023
@kodiakhq kodiakhq Bot deleted the fix/install-zoho-crm branch September 6, 2023 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge core area: core, team members only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants