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 session #8

Merged
merged 4 commits into from
Feb 10, 2021
Merged

Add session #8

merged 4 commits into from
Feb 10, 2021

Conversation

ssoper
Copy link
Owner

@ssoper ssoper commented Feb 10, 2021

  • Create a Session object which can be passed to endpoint APIs
  • Breakup Etrade class into smaller classes organized around the E*TRADE API endpoints
  • Organize under a new connectors.etrade package
  • Cache retrieved tokens + verifier code and attempt to re-use via the Renew Token endpoint vs. making another Request Token roundtrip

@ssoper ssoper merged commit 43d6777 into master Feb 10, 2021
@ssoper ssoper deleted the features/add-session branch February 10, 2021 01:27
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.

1 participant