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

feat: add coin price plugin #1808

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

proteanx
Copy link
Contributor

@proteanx proteanx commented Jan 4, 2025

Relates to

#1773 & #1382

Risks

Low risk, minimal changes and a very simple plugin

Background

This does kind of replace both #1773 & #1382. I included 1773's commits but 1382 was too different and too many commits to include. This plugin adds 3 new price data APIs. It first looks for Coingecko API key, then CoinMarketCap & and if neither are present it falls back to CoinCap.io's free API.

What does this PR do?

Adds price retrieval via the GET_PRICE action.

image

What kind of change is this?

Features (non-breaking change which adds functionality)

Documentation changes needed?

My changes do not require a change to the project documentation.

Testing

Where should a reviewer start?

Review plugin files (they are short and simple)

Detailed testing steps

Add an API key if you have one (check .env.example), if not it will default to CoinCap.io API (which does have less coins but works well). CoinGecko API is recommended, CoinMarketCap also an option. All confirmed to be working.

Discord username

proteanx

@proteanx proteanx changed the title (feat) add coin price plugin feat: add coin price plugin Jan 4, 2025
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