crisp-im / ruby-crisp-api

💎 Crisp API Ruby Wrapper
https://docs.crisp.chat/guides/rest-api/
MIT License
16 stars 7 forks source link

New idea: Add `page_size` query parameter #18

Closed ydakuka closed 1 year ago

ydakuka commented 1 year ago

In order not to deal with rate limits, please add page_size query parameter (default, 20).

eliottvincent commented 1 year ago

Hello, we don't plan to implement this.

ydakuka commented 1 year ago

@eliottvincent okay, what are the rate limits? Could you please add more info to the developer guide? I don't know what timeout should I add when I make 1000000+ requests due to page_size = 20.

eliottvincent commented 1 year ago

We don't disclose our rate-limits, as they are dynamic and different according to every API route.

Just re-schedule your API calls when you hit a 422 or 429 HTTP response.

ydakuka commented 1 year ago

There's a headache here, but okay.