search

deanishe / alfred-convert

Convert between different units in Alfred
Other
721 stars 56 forks source link

Improve handling of Currency update responses #82

Open cameronsstone opened 3 years ago

cameronsstone commented 3 years ago

There are two commits here:

  1. Generically reject any currency updates unless both are true:
    1. the currency is in CURRENCIES or CRYPTO_CURRENCIES
    2. the rate is a number.
  2. Explicitly handle throttling responses from Cryptocompare.com's API which unfortunately use a HTTP 200 instead of a 429 or similar.

This should help with #74 .