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

Implementations proxy update #67

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

shriyatyagii
Copy link
Member

Created separate implementations for each route and added all the necessary route functions inside it.

Working -
The proxy will call the Main Implementation. The Main implementation will do a delegate call to that particular route's implementation. All execution logic resides in the new implementations. Additionally, all the new Implementations of the same chain share a common helper, events and variables file.

Changes -

  1. Created implementations for each route
  2. Created a common folder that contains common helpers and interfaces of all chains.
  3. Added additional function spell in common helpers
  4. Removed internal functions from Main implementations
  5. Added spell calls inside each external flashloan function
  6. Added new test cases file for each chain

Test Result-
3021CF6C-33A7-41C4-86DC-DED09AB78964

@KaymasJain KaymasJain requested a review from Vaibhav20201 May 22, 2022 15:43
@Sowmayjain Sowmayjain removed the request for review from Vaibhav20201 December 11, 2022 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants