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

Merge main onto crypto for cleanliness in preparation for future merge onto main #12

Merged
merged 96 commits into from
Feb 16, 2023

Conversation

AbhishekRaoC
Copy link
Collaborator

No description provided.

AFaverooo and others added 30 commits January 28, 2023 18:12
Change the redirect URL in link token creation method
Create investments class to connect to the plaid API
Create connect_investments view to pass the link token to JavaScript
Create connect_investments html page to initialise plaid link's user setup
Ensure development_wrapper properly implements the plaid_wrapper interface
Remove plaid sandbox api functionality from views
Change the connect_investments view to use methods specified by the plaid_wrapper interface
Change create_link_token in plaid_wrapper to request institutions from all possible countries
sminakov and others added 29 commits February 8, 2023 19:18
Move the save_access_token and getter methods to the plaid_wrapper superclass
Refactor tests
…er class

Add the retrieve_access_tokens method to plaid_wrapper
Refactor SandboxWrapper and remove duplicate methods inherited from superclass
Add tests for new methods in stocks.py
Refactor other tests
Create a view to link sandbox investment data to a logged in user
Create a method to reformat investment data to just name and total price in StockGetter
Create a view to display reformatted stock data
@AbhishekRaoC AbhishekRaoC merged commit 0db68e5 into crypto Feb 16, 2023
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.

5 participants