Skip to content
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

Add new Gmail API Service Account Source type #26

Merged
merged 7 commits into from Jun 21, 2021

Conversation

chadell
Copy link
Contributor

@chadell chadell commented Jun 15, 2021

TODO

  • Create an abstract EmailSource class that will concentrate some methods and attributes valid either for IMAP or Gmail API. This is more concrete than the Source that could eventually fetch directly REST API with JSON data (for instance)
  • Validate that the API calls works once we could implement "Delegate domain-wide authority"
  • Implement the Retrieve emails logic and looks for synergies with the IMAP method to simplify both
  • Define how the Service Account works and all the steps the user has to do in Google to make it work to add to Readme
  • Extend testing for the new class

image

@chadell chadell force-pushed the issue15-add-gmail-api-source branch from 3a76940 to 70548f6 Compare June 16, 2021 09:44
@chadell chadell force-pushed the issue15-add-gmail-api-source branch from 70548f6 to 4762ca4 Compare June 16, 2021 09:51
@chadell
Copy link
Contributor Author

chadell commented Jun 16, 2021

@glennmatthews , this PR still needs to add documentation on setting this up (Jonathan will help me with as there is some admin stuff to be done in Google console) and include testing to new methods.
But, while I'm finishing with that, maybe a first glance of the PR could be useful...

@chadell chadell changed the title [WIP] Add new Gmail API Service Account Source type Add new Gmail API Service Account Source type Jun 16, 2021
@chadell chadell marked this pull request as ready for review June 16, 2021 15:42
@chadell
Copy link
Contributor Author

chadell commented Jun 16, 2021

@glennmatthews , taking into account that I still have to update Readme with the instructions to use the new Notification Source, the PR is decently ready for review

@chadell chadell force-pushed the issue15-add-gmail-api-source branch from dbc7555 to b86e5f0 Compare June 16, 2021 15:44
@LoneStar-Swish
Copy link
Contributor

I added on to the API instructions and made a couple small grammar changes. If I need to expand on anything let me know.

glennmatthews
glennmatthews previously approved these changes Jun 18, 2021
README.md Outdated Show resolved Hide resolved
Co-authored-by: Glenn Matthews <glenn.matthews@networktocode.com>
@chadell chadell merged commit 87323df into develop Jun 21, 2021
@chadell chadell deleted the issue15-add-gmail-api-source branch June 21, 2021 05:23
itdependsnetworks pushed a commit to itdependsnetworks/nautobot-app-circuit-maintenance that referenced this pull request Apr 3, 2024
…autobot#26)

* Refactor library to allow a Provider to concatenate multiple parsers
* Remove unused sender, source and subject attributes
Co-authored-by: Glenn Matthews <glenn.matthews@networktocode.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants