Skip to content

Provide access to the Access Tokens API and enable initialization from OAuth Authentication Codes and Refresh Tokens #125

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jul 13, 2020

Conversation

cmlccie
Copy link
Collaborator

@cmlccie cmlccie commented Jul 9, 2020

@sQu4rks, I know you added support for #61 by extending the initializer for the WebexTeamsAPI class. What do you think of this alternative solution?

This PR provides access to the access_tokens API wrapper directly from the WebexTeamsAPI class before instantiating an instance of the class. Also, provides convenience class methods that will instantiate a new WebexTeamsAPI connection object from an OAuth Authorization Code or Refresh Token.

Example Accessing the Access Tokens API:

access_token_object = WebexTeamsAPI.access_tokens.get(client_id, client_secret, code, redirect_uri)
access_token_object = WebexTeamsAPI.access_tokens.refresh(client_id, client_secret, refresh_token)

Example Create WebexTeamsAPI Connection Objects from OAuth Access Tokens or Refresh Tokens:

api = WebexTeamsAPI.from_oauth_code(client_id, client_secret, code, redirect_uri)
api = WebexTeamsAPI.from_oauth_refresh(client_id, client_secret, refresh_token)

@cmlccie cmlccie requested a review from sQu4rks July 9, 2020 20:34
@cmlccie cmlccie changed the base branch from master to v1.x July 9, 2020 20:35
@cmlccie cmlccie linked an issue Jul 9, 2020 that may be closed by this pull request
Provide access to the access_tokens API wrapper directly from the WebexTeamsAPI class *before* instantiating an instance of the class (which requires an access token).

Also provide convenience class methods that will instantiate a new WebexTeamsAPI connection object from an OAuth Authorization Code or Refresh Token.
@sQu4rks
Copy link
Contributor

sQu4rks commented Jul 10, 2020

I like the idea! Let me review the code a bit more in-depth.

@sQu4rks sQu4rks merged commit 496507a into v1.x Jul 13, 2020
@sQu4rks
Copy link
Contributor

sQu4rks commented Jul 13, 2020

Happy with these changes. I'll merge the PR.

@cmlccie cmlccie deleted the feature-61 branch February 19, 2024 19:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ability to initialise API object using OAUTH details
2 participants