-
-
Notifications
You must be signed in to change notification settings - Fork 643
Open
Labels
π» discussionPriority: HighHigh priority to include it inside next releaseHigh priority to include it inside next release
Description
Dear contributors
Consider that commits such as this:
Really should not be merged as they introduce breaking functionality silently. As it happened for us - breaking production for any users with degraded internet.
5 seconds very very short for mobile clients, so at a minimum it should have been immediayely configurable. Preferably just nullable until set.
Due to this commit, it was set to 5 seconds, i reckon left from development, and now all downstream commits have all tagged along, setting the default timeout to 5 seconds.
Such as
Thank you for contributions otherwise - love the library π got very spooked from this
Please consider this for the futuree, and let us close this issue again
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
π» discussionPriority: HighHigh priority to include it inside next releaseHigh priority to include it inside next release