Skip to content
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

Store bridged token price in the DB #3667

Merged
merged 2 commits into from
Mar 1, 2021

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Feb 26, 2021

Motivation

Bridge market cap updater can not get token price reliably from CoinGecko for the whole bridged tokens list

Changelog

Save exchange rate of token to the DB and gets old value from DB, if it is nil or 0 at CoinGecko scheduled update.

Checklist for your Pull Request (PR)

@coveralls
Copy link

coveralls commented Feb 26, 2021

Pull Request Test Coverage Report for Build e3d0eb2109af1fbba2cab1b1789905b14f5cf82b-PR-3667

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.0%) to 64.8%

Totals Coverage Status
Change from base Build c047c212f6ec595db2afe3fcea9da0ac2333103b: 1.0%
Covered Lines: 2106
Relevant Lines: 3250

💛 - Coveralls

@vbaranov vbaranov force-pushed the vb-bridged-tokens-exchange-rate-in-db branch from e3d0eb2 to 771078f Compare March 1, 2021 09:40
@vbaranov vbaranov merged commit 7d9b166 into master Mar 1, 2021
@vbaranov vbaranov deleted the vb-bridged-tokens-exchange-rate-in-db branch March 1, 2021 12:25
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.

2 participants