-
Notifications
You must be signed in to change notification settings - Fork 30
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
Feat/rest api upgrade #1167
Feat/rest api upgrade #1167
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #1167 +/- ##
===================================================
+ Coverage 59.08061% 59.17362% +0.09300%
===================================================
Files 538 565 +27
Lines 32217 33302 +1085
Branches 275 275
===================================================
+ Hits 19034 19706 +672
- Misses 11479 11797 +318
- Partials 1704 1799 +95
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
` | ||
<h1>Invalid Params</h1> | ||
<hr/> | ||
<b>Ensure that your request matches the following format: /swap?chain=1&fromToken=USDC&toToken=DAI&amount=100</b> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably want /swapTxInfo
instead of /swap
here right?
LGTM |
This reverts commit f8efa3d.
Description
This upgrade reconfigures dependencies so that it is compatible with heroku deployment, adds two new functions that allow users to generate the raw transaction data in python (so they can acutually bridge once connected to Web3), and updates the ReadMe.
Additional context
Ideally @nautsimon @dwasse can review the two new functions (very slight changes) to confirm that the output transaction data is correct. This also probably needs more robust testing
Metadata