Skip to content

Google oauth support for Google calender tool #7788

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

Merged

Conversation

deepu105
Copy link
Contributor

@deepu105 deepu105 commented Mar 1, 2025

  • Added access token support via Google OAuth
  • Added keyfile support for consistency with Gmail tool
  • Fix prompts since output was not in correct format

Copy link

vercel bot commented Mar 1, 2025

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

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Mar 1, 2025 7:30pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Mar 1, 2025 7:30pm

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. auto:improvement Medium size change to existing code to handle new use-cases labels Mar 1, 2025
@deepu105
Copy link
Contributor Author

deepu105 commented Mar 2, 2025

@benjamincburns let me know if there is anything to be addressed here. I assumed the tool can be used in a singleton as well so made the access token a string or function so that token refresh and validation can be handled on the agent side

@dosubot dosubot bot added the lgtm PRs that are ready to be merged as-is label Mar 4, 2025
@benjamincburns
Copy link
Collaborator

Looks great - I appreciate the ability to set the accessKey using a function - should make this very flexible. Thanks for the contribution @deepu105!

@benjamincburns benjamincburns merged commit 8f783fd into langchain-ai:main Mar 4, 2025
32 checks passed
@deepu105 deepu105 deleted the google-oauth-support-calender branch March 5, 2025 04:38
FilipZmijewski pushed a commit to FilipZmijewski/langchainjs that referenced this pull request Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto:improvement Medium size change to existing code to handle new use-cases lgtm PRs that are ready to be merged as-is size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants