connext / monorepo

Connext is a modular stack for trust-minimized, generalized communication between blockchains.
https://docs.connext.network
MIT License
292 stars 173 forks source link

Fall back to our own estimate fee calculation and share between sequencer and SDK #3029

Closed rhlsthrm closed 1 year ago

rhlsthrm commented 1 year ago

Problem

What problem are we solving and why does it matter, ser? SDK and sequencer calculate the relayer fee separately. Also in some cases our external API returns 0.

Proposed Solution

Thoughts on solution design. We should do to solve this problem.

Acceptance Criteria

From an end tester and/or user perspective, the below criteria must be met to consider this done:

wanglonghong commented 1 year ago

We were depending on our own price oracle contract to implement fallback mechanism but it looks like we don't use it on amarok

rhlsthrm commented 1 year ago

Since we only use gas tokens, we can use off the shelf APIs such as CoinGecko for this. Maybe we should have a few that we cycle between.