-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
feat: Gmail Loader component created #3241
feat: Gmail Loader component created #3241
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
ran make format and ran make lint
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.
How are you?
Thanks for this component. It is looking good. I've left some comments.
Errors Resolved. Suggestion from @ogabrielluiz added. Note this is a ChatLoader, since GmailLoader uses the BaseChatloader as base class. But this would be a highly useful feature. if required i could rewrite the component as document loader. but either way this would be an amazing feature. Planning to publish gmails tools also in next steps mostly.
Hi @ogabrielluiz We are using this component as well as the Google Drive component for our internal project as custom components. Hence thought of contributing these components back to the community. |
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.
Awesome!
LGTM
Key Changes: 1. Added Optional to the import statement. 2. Updated the type hint of label_ids to Optional[List[str]] to allow it to be None or a list of strings.
Added Custom Gmail Chat loader to load emails using service account token as json string
Requesting review:
@jordanrfrazier @nicoloboschi