Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

feat: add transaction price information to meta-txn endpoints for UI #248

Merged
merged 6 commits into from
Jun 11, 2020

Conversation

fragosti
Copy link
Contributor

@fragosti fragosti commented Jun 8, 2020

Description

We want to communicate exactly how much money we're saving users in the meta-txn endpoint too. This is a bit different than the regular endpoint:

  • We don't need to tell them about the refund they would potentially be getting with gas tokens.
  • The estimates don't need to be as accurate.
  • We can use protocolFee instead of minimumProtocolFee.

Testing Instructions

Checklist

  • Update documentation as needed. Website Documentation PR:
  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.

@dekz dekz force-pushed the feat/estimated-fees-meta branch from 80ffdfc to f11cd3e Compare June 10, 2020 06:54
@dekz dekz force-pushed the feat/estimated-fees-meta branch from f11cd3e to c255b83 Compare June 10, 2020 07:28
@dekz
Copy link
Member

dekz commented Jun 10, 2020

deploy staging

@dekz dekz force-pushed the feat/estimated-fees-meta branch from a597cef to 7e4a0f9 Compare June 10, 2020 10:02
@dekz dekz force-pushed the feat/estimated-fees-meta branch from 7e4a0f9 to 0b13670 Compare June 10, 2020 10:12
@dekz
Copy link
Member

dekz commented Jun 10, 2020

deploy staging

@dekz dekz merged commit d26d6c6 into master Jun 11, 2020
@dekz dekz deleted the feat/estimated-fees-meta branch June 11, 2020 05:52
github-actions bot pushed a commit that referenced this pull request Jun 15, 2020
# [1.8.0](v1.7.0...v1.8.0) (2020-06-15)

### Bug Fixes

* uniswap v2 buys ([#253](#253)) ([225db22](225db22))

### Features

* Add support for UniswapV2 and MultiBridge ([#250](#250)) ([29636e7](29636e7)), closes [#252](#252)
* add transaction price information to meta-txn endpoints for UI ([#248](#248)) ([d26d6c6](d26d6c6))
@github-actions
Copy link

🎉 This PR is included in version 1.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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