Skip to content
This repository has been archived by the owner on Jul 9, 2021. It is now read-only.

Streamline SwapQuoter and remove instant-only functions #2727

Closed
wants to merge 3 commits into from

Conversation

xianny
Copy link
Contributor

@xianny xianny commented Oct 8, 2020

Description

  • Streamline SwapQuoter by passing MarketOperation as a param instead of having separate functions for buy/sell
  • Delete functions that are only used by Instant

Testing instructions

Types of changes

Checklist:

  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • Add new entries to the relevant CHANGELOG.jsons.

@xianny xianny changed the base branch from asset-swapper-v1-refactor to refactor-asset-swapper-v1 October 8, 2020 00:28
@xianny xianny force-pushed the refactor/AS-fewer-functions branch from 30320ce to 4d2264c Compare October 8, 2020 00:39
@xianny xianny requested review from abandeali1 and hysz as code owners October 8, 2020 00:39
@alexkroeger alexkroeger force-pushed the refactor-asset-swapper-v1 branch from b1e2f44 to 1adb56f Compare October 8, 2020 23:44
Base automatically changed from refactor-asset-swapper-v1 to development October 9, 2020 16:55
@stale
Copy link

stale bot commented Nov 15, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 15, 2020
@stale
Copy link

stale bot commented Dec 19, 2020

This issue has been automatically closed because no activity occured in 7 days after being marked as stale. If it's still relevant - feel free to reopen. Thank you for your contributions.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants