-
Notifications
You must be signed in to change notification settings - Fork 7
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
1.6.1-0.1.2: Enhancement - Exchange Rate API #152
Conversation
I also updated the Sveltekit dependency to fix a couple of security alerts. |
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Ignoring: Pull request alert summary
📊 Modified Dependency Overview:
|
@SocketSecurity ignore @sveltejs/kit@1.15.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
had a good play around, dont see any issues! app actually feels a little faster or is it just me??
This PR connects Clams to a new server endpoint:
https://api.clams.tech
which has a new codebase. Exchange rates will be fetched and cached by the server, and the app will fetch from there instead. This will help with rate limits on the external exchange rate API's.To save bandwidth the app will only fetch the exchange rate for the fiat denomination that is set in the app. A new exchange rate will be fetched every 5 minutes or when the fiat denomination is changed.
The wsproxy lives at the /ws-proxy route on the new server and Clams now connects to that.
The https proxy for lnurl calls lives at /http-proxy.
Closes #144