mobilecoinofficial / forest

Enables a Forest of MobileCoin enabled SignalBots. Beta software, APIs may change!
MIT License
19 stars 12 forks source link

Try to wait before sending more than 60 messages/min and retry sends on error #82

Closed technillogue closed 2 years ago

technillogue commented 2 years ago

yay

┆Issue is synchronized with this Asana task by Unito

sync-by-unito[bot] commented 2 years ago

➤ Sylvie Liberman commented:

will close Investigate SignalApp rate limiting ( https://app.asana.com/0/1201315333674405/1201315333674435/f ) when merged

iamalwaysuncomfortable commented 2 years ago

The if else can be slight restructured, but the goal roughly is to have the ability to NOT log transactions into the full service DB if the sender wants it private. Putting the account_id into the request logs it to full service, if you omit it, it doesn't get logged. The if else is basically to give the bot author options to not log it if they don't want.

technillogue commented 2 years ago

The if else can be slight restructured, but the goal roughly is to have the ability to NOT log transactions into the full service DB if the sender wants it private. Putting the account_id into the request logs it to full service, if you omit it, it doesn't get logged. The if else is basically to give the bot author options to not log it if they don't want.

cool, sounds good - can you add a comment with that?