Skip to content
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

Add default and minimum values for operation polling + document it #150

Closed
kober32 opened this issue Mar 12, 2024 · 1 comment
Closed

Add default and minimum values for operation polling + document it #150

kober32 opened this issue Mar 12, 2024 · 1 comment
Assignees

Comments

@kober32
Copy link
Member

kober32 commented Mar 12, 2024

Right now, polling intervals can be set to any number, which can be a problem for the server if it's too frequent.

We should introduce a default value of 7 and set the minimum time to 5 seconds to prevent extreme server overload that can be caused if the value is set to very small numbers.

Also, we should be more explicit about it in the documentation and explain how to minimize the impact on the server (like when push notifications are enabled, the interval should be doubled to prevent server overload).

@kober32 kober32 changed the title Add default and minimum values for operation polling + document Add default and minimum values for operation polling + document it Mar 12, 2024
@kober32
Copy link
Member Author

kober32 commented Mar 13, 2024

How to achieve this:

  • extension method that does not require an interval
  • when the interval is lower than 5, we should change it to 5 and log!! this event
  • In the documentation section, add a text where we advise that the optimal time is 7s and 14s when push notifications are allowed (copy the example from our app).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants