Give quotes for inbound channels and email #11
Conversation
|
Running this in prod already since the nolooking PR is more important |
nickfarrow
force-pushed
the
inbound
branch
3 times, most recently
from
fc2ddf4 to
2a8616c
Compare
|
please consider using POST as changed in this commit DanGould@362fdde. That commit is now running in prod |
|
Shouldn't this only send a notification when someone pays us? Otherwise we have to watch the payment address |
nickfarrow
force-pushed
the
inbound
branch
2 times, most recently
from
3d16959 to
3fe03c5
Compare
| let price = 30_000; | ||
| let resp_capacity = 1_000_000; //sats | ||
| let resp_duration = 1; //month | ||
| let payment_address = crate::lnd::get_onchain_address().await; |
There was a problem hiding this comment.
It seems a bit hacky to have gotten the software this far and still call what look like static functions when they that rely on a specific config. Still good on you for making it work
There was a problem hiding this comment.
I think fine, the config is loaded within each function. nicer than keeping config around
|
TODO: Bubble up results within that separate thread and just ignore them. We can't afford to panic! |
|
I think this is what we're running. If so, it gets my approval and should be merged asap |
Adds an endpoint
Resposnds with an address.
Currently hardcoded to
Sends us an email notification/invoice using aws ses via
(need
~/.aws/credentials)