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

dev : add pragma api to compute fees in STRK #1633

Merged
merged 19 commits into from
Jun 21, 2024
Merged

dev : add pragma api to compute fees in STRK #1633

merged 19 commits into from
Jun 21, 2024

Conversation

azurwastaken
Copy link
Contributor

Pull Request type

  • Feature

What is the current behavior?

STRK gas price are computed with a constant eth/strk price

Resolves: #NA

What is the new behavior?

  • use pragma API to fetch real time eth/strk price to compute real gas fees

Does this introduce a breaking change?

No

Other information

🚨Not mergeable yet 🚨
This should work, however i don't know how to handle api key. I was planning to put it in a config file but i struggle ton handle this within the code.

crates/client/l1-gas-price/src/worker.rs Outdated Show resolved Hide resolved
crates/client/l1-gas-price/src/worker.rs Outdated Show resolved Hide resolved
crates/client/l1-gas-price/src/worker.rs Outdated Show resolved Hide resolved
examples/messaging/eth-config.json Outdated Show resolved Hide resolved
examples/messaging/eth-config.json Outdated Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
@EvolveArt EvolveArt requested a review from tdelabro June 20, 2024 11:55
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/l1-gas-price/src/worker.rs Show resolved Hide resolved
crates/client/eth-client/src/config.rs Outdated Show resolved Hide resolved
crates/client/l1-gas-price/src/worker.rs Outdated Show resolved Hide resolved
@EvolveArt EvolveArt requested a review from tdelabro June 21, 2024 00:07
@tdelabro tdelabro merged commit 7b40592 into keep-starknet-strange:main Jun 21, 2024
15 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants