-
Notifications
You must be signed in to change notification settings - Fork 5.5k
[Gmail] Custom OAuth Components #4125
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
LGTM!
_client() { | ||
const auth = new gmail.auth.OAuth2(); | ||
auth.setCredentials({ | ||
access_token: this.$auth.oauth_access_token, |
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.
Move this to a getter method.
access_token: this.$auth.oauth_access_token, | |
access_token: this._accessToken(), |
Testing is skipped since there are some prerequisites required for testing. @andrewjschuang please ship it and @dannyroosevelt will perform the testing. |
* master: (22 commits) Adding app scaffolding for gmail Adding app scaffolding for gmail Modifying global config Adding app scaffolding for crisp Adding app scaffolding for crisp Adding app scaffolding for crisp Modifying global config Gmail Custom OAuth minor fixes (#4126) [Gmail] Custom OAuth Components (#4125) Fixed postgres event (#4115) Adding app scaffolding for gmail_custom_oauth Adding app scaffolding for gmail_custom_oauth Adding app scaffolding for gmail_custom_oauth Modifying global config Update send-large-message.mjs TimeCamp #3812 (#4084) Small updates to Slack components (#4108) add default false value (#4117) Sources - ClickUp (#4101) Esputnik - new sources and actions (#4105) ...
Adds components for new Gmail Custom OAuth app.
Note: the app is currently hidden from the UI, so I wasn't able to test them.
Resolves #4121.