-
Notifications
You must be signed in to change notification settings - Fork 125
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
create a euro to USD parser #276
Comments
if we integrate that api we have only 250 limit per month right? |
Yes, though I would be shocked if we run into that limit. I don't think it should happen every time. I suggest trigger with a 💶 or 💵 reaction. |
just make sure to have a failure path so the bot can explain why it didn't run |
Why not just leverage the oob currency conversion? There are only 5 currencies active oob, but we could turn on the full list and the exchange rates update nightly. |
Maybe Chris is using Slacker for forex trades and needs real time data.
…On Wed, Oct 2, 2024, 01:31 eriemer1 ***@***.***> wrote:
Why not just leverage the oob currency conversion?
if you add a record to fx_currency_instance with the amount and currency
and set the reference currency to the one you want to convert to it will
automatically fill in the reference amount with the converted value.
There are only 5 currencies active oob, but we could turn on the full list
and the exchange rates update nightly.
—
Reply to this email directly, view it on GitHub
<#276 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADBGRXJYLTDKOFQE6OLADRDZZOAKNAVCNFSM6AAAAABO5P3HGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBXGY3DENJXGQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
did you mean if we ran out free service then we just show the currency emoji? |
No I mean a reaction using the emoji is used to trigger the bot, rather
than it being a text based trigger.
…On Wed, Oct 2, 2024, 08:02 K Narasimha Reddy ***@***.***> wrote:
Yes, though I would be shocked if we run into that limit.
I don't think it should happen every time. I suggest trigger with a 💶 or
💵 reaction.
did you mean if we ran out free service then we just show the currency
emoji?
—
Reply to this email directly, view it on GitHub
<#276 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADBGRXMTR6NY42QFORB6AJ3ZZPOG7AVCNFSM6AAAAABO5P3HGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBYGQ3TANBRG4>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
That’s a good point. And with the right api we could include crypto
|
related to #293 |
https://exchangeratesapi.io/
should it happen automatically like the C to F parser? 🤷♀️
The text was updated successfully, but these errors were encountered: