Skip to content

Task/sean/str 698 #234

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

Merged
merged 26 commits into from
Jul 19, 2023
Merged

Task/sean/str 698 #234

merged 26 commits into from
Jul 19, 2023

Conversation

auroter
Copy link
Contributor

@auroter auroter commented Jul 18, 2023

Created an internal Gas Rate calculator for networks that Owlracle does not support (such as DFK Subnet). This will eventually be used in conjunction with a gas analysis service that determines the boost and tip % for a given network at the current moment in time.

To test:

  1. Get a quote in Postman on Fuji Testnet (Anything with 43113)
  2. Note the GasUSD in the quote
  3. Go into Postico and set the Fuji network Gas Oracle Name to "internal":
image 4) Get a quote again
  1. Note that the GasUSD was still estimated (The value may be different than before - as Fuji was using the Avax gas owlracle)

Sean and others added 26 commits July 2, 2023 15:30
…ution of multiple calls, added calls for NFT and Token Forwarding (not complete yet)
…cko always appeared to be down, fixed an issue with updating the timestamp of a cached token value even when failing to retrieve a new value
…d a bug where finding 0 tokens to forward for ERC721 or ERC20 reported an error, fixed a bug where forwards failed because they were using safeTransferFrom without an approve, added true max gas caching for each transaction request, fixed a bug where forwarding would be attempted when there was nothing to forward, added true gas summing including forwarding
…s during Cost estimation if network oracle name is set to "internal" in the db. Also use this value when generating the real evm transaction request.
Copy link

@brianspierce brianspierce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@auroter auroter merged commit 3b7ed9e into develop Jul 19, 2023
@auroter auroter deleted the task/sean/str-698 branch July 19, 2023 18:15
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.

4 participants