Skip to content

throttle interval fIx for android#33

Closed
leonidovia wants to merge 1 commit intomeg4cyberc4t:mainfrom
leonidovia:patch-1
Closed

throttle interval fIx for android#33
leonidovia wants to merge 1 commit intomeg4cyberc4t:mainfrom
leonidovia:patch-1

Conversation

@leonidovia
Copy link
Copy Markdown

Кажется, что тут вообще нет смысла задавать в ms, раз на обеих платформах принимается в sec.
А ещё было бы здорово на стороне Flutter сделать Duration вместо int

@meg4cyberc4t
Copy link
Copy Markdown
Owner

Это не совсем так. Согласно официальной документации:

Android:

.withThrottleInterval(2000) — ограничение частоты обновления конфига в миллисекундах. Рекомендуется использовать только для тестирования.

iOS:

.fetchThrottle — ограничение частоты обновления конфига в секундах. Рекомендуется использовать только для тестирования.


Изменения fetchThrottleIntervalMs на использование Duration я одобряю, это действительно будет чуть более понятно для разработчика.

@leonidovia
Copy link
Copy Markdown
Author

Это не совсем так. Согласно официальной документации:

Эта документация устарела, если посмотреть внутрь withThrottleInterval - там даже параметр называется fetchThrottleIntervalSec, ну и если проверить - допустим, передать 1000 - то пока не пройдёт именно 1000 секунд, будем получать исключение о троттлинге

@meg4cyberc4t
Copy link
Copy Markdown
Owner

meg4cyberc4t commented Aug 12, 2025

@leonidovia Спасибо за внимательность!
Вернулся к этому сейчас, когда документация по нативным библиотекам была обновлена.
Добавил этот функционал в #39

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

Successfully merging this pull request may close these issues.

2 participants