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

chore: add maga to coingecko #538

Merged
merged 5 commits into from
Jun 24, 2024
Merged

chore: add maga to coingecko #538

merged 5 commits into from
Jun 24, 2024

Conversation

nivasan1
Copy link
Contributor

@nivasan1 nivasan1 commented Jun 21, 2024

In This PR

  • I add the maga/usd coingecko feed to the coingecko markets
  • This will purely be used for price-verification in our dev environments
  • I've tested this locally via make start-all-dev and the price is inline w/ the analogous CG feed

@nivasan1 nivasan1 added preview Select this label when you want to preview a test chain + sidecar deployment in K8S vpn Select this label when you want to proxy all traffic in the deployment through a VPN. marketmap_provider=marketmap_api Select this label if you want to spin up a sidecar + simapp where the two communicate. coingecko Select this label if you want the deployment to include coingecko markets. Only used for sim app. labels Jun 21, 2024
Copy link

codecov bot commented Jun 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@bff74f6). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #538   +/-   ##
=======================================
  Coverage        ?   56.60%           
=======================================
  Files           ?      244           
  Lines           ?    12288           
  Branches        ?        0           
=======================================
  Hits            ?     6956           
  Misses          ?     4644           
  Partials        ?      688           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nivasan1 nivasan1 added the raydium_markets Select this label if you want the deployment to include Raydium markets. Only used for sim app. label Jun 21, 2024
@MaghnusM MaghnusM closed this Jun 21, 2024
@MaghnusM
Copy link
Contributor

Unclear why this is urgent or needed right now (wouldn't this just be a config change on the chain side/marketmap side?), need more explanation.

@MaghnusM MaghnusM reopened this Jun 21, 2024
@MaghnusM
Copy link
Contributor

Reopening

Comment on lines 6233 to 6242
{
"name": "coingecko_api",
"off_chain_ticker": "ponke/usd"
}
]
},
"USA/USD": {
"ticker": {
"currency_pair": {
"Base": "USA",
Copy link
Contributor

Choose a reason for hiding this comment

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

can you pipe this into a json formatter

@davidterpay
Copy link
Contributor

kek

@nivasan1 nivasan1 force-pushed the nv/add-trump-cg-market branch from 9c4f787 to c053a9b Compare June 24, 2024 19:06
@nivasan1 nivasan1 merged commit 9f0c32a into main Jun 24, 2024
16 checks passed
@nivasan1 nivasan1 deleted the nv/add-trump-cg-market branch June 24, 2024 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coingecko Select this label if you want the deployment to include coingecko markets. Only used for sim app. marketmap_provider=marketmap_api Select this label if you want to spin up a sidecar + simapp where the two communicate. preview Select this label when you want to preview a test chain + sidecar deployment in K8S raydium_markets Select this label if you want the deployment to include Raydium markets. Only used for sim app. vpn Select this label when you want to proxy all traffic in the deployment through a VPN.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants