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

Long Short Pair Support #88

Merged
merged 21 commits into from
Nov 8, 2021
Merged

Long Short Pair Support #88

merged 21 commits into from
Nov 8, 2021

Conversation

snake-poison
Copy link
Contributor

@snake-poison snake-poison commented Nov 1, 2021

Description of changes proposed in this pull request:

  • Modifies types to support alternatives to the EMP financial contracts
  • Modifications to the asset class to support LSP
  • Incorporates a mutlicall provider through the @0xsequence/multicall
  • Basic test to test LSPState
  • Removed the need to import our own abi as the UMA package contains all of the interfaces that we need so far.

Checks:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submission:

  • Have you successfully ran tests with your changes locally?
  • Have you lint your code locally prior to submission?

Changes to Existing Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, if applicable?

Modified Types and assets class to support FinancialContracts( LSP or EMP ) and Assets (LSP or EMP)
Uses multicall through the ethcall package to reduce the amount of network connections required to obtain state.
@ghost ghost mentioned this pull request Nov 1, 2021
src/assetstest.json Outdated Show resolved Hide resolved
src/lib/Asset.ts Outdated Show resolved Hide resolved
Replaced ethcall package with @0xsequence/multicall
Added cross-env package.
Updated packages to their latest version except for typechain which could only be upgraded to 5.2.0 as ^6 breaks the type generation for the project.
hardhat.config.ts Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
src/lib/Asset.ts Outdated Show resolved Hide resolved
src/lib/Asset.ts Show resolved Hide resolved
@ghost
Copy link

ghost commented Nov 5, 2021

@ElmoArmy I suggest to merge develop into feat-add-lsp.

@snake-poison
Copy link
Contributor Author

@ElmoArmy I suggest to merge develop into feat-add-lsp.

You got it. Done in c2b0a78

@snake-poison snake-poison self-assigned this Nov 5, 2021
@ghost ghost mentioned this pull request Nov 8, 2021
5 tasks
@snake-poison snake-poison marked this pull request as ready for review November 8, 2021 12:51
@snake-poison snake-poison merged commit 50de22d into develop Nov 8, 2021
@ethedev ethedev deleted the feat-add-lsp branch November 15, 2021 13: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