-
Notifications
You must be signed in to change notification settings - Fork 929
Documentation is using a small minimumFetchIntervalMillis without explicitly specifying that #2841
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
Comments
I found a few problems with this issue:
|
The same problem on Firefox Browser 82.0.3 |
We are also having this issue on Firefox. What should one supposed to do when this message comes? Increase the fetchTimeoutMillis? In our case it is already 1 minute. |
@fischermatte @yoldar I don't believe this issue is related to firefox specifically. The issue here is the value used in documentation 3600000 (1 hour) is extremely low for production environment. What's even worse is what is written makes you think this is a recommended production value, i am referring to this sentence "During development, it's recommended to set a relatively low minimum fetch interval. See Throttling for more information." The production value is documented in https://firebase.google.com/docs/remote-config/use-config-web#throttling which is 12 hours. You should still catch this error, and try to fetch config in a longer interval. But once you use the default production value (12 hours) i doubt you will see it. |
Thx @sanehab. Ok I guess my problem might not be well placed here. I am just looking for an explanation why this error pops up at load time on firefox but not on other engines. Those parameters shouldn't have an impact at initial load time with a fresh-cache-cleaned browser. |
Still having this problem on Firefox. Does someone has any fix for this bug? |
I also have an issue with Firefox only. Automatically it doesn't load anything (for some reason), and when I call PS: Firefox version 120.0.1 |
This solution helped for me, I added a
|
Can we implement ‘real-time’ on the web? Or, the remote configuration server should use a caching mechanism like ETag. Naturally, the web page should fetch the configuration with each page request. |
…valMillis. Issue addressed: firebase/firebase-js-sdk#2841
…illis (#389) * Add comment to highlight the default prod value for minimumFetchIntervalMillis. Issue addressed: firebase/firebase-js-sdk#2841 * Update code snipped with default prod value for minimumFetchIntervalMillis. * Update code snipped with default prod value for minimumFetchIntervalMillis. * Update code snipped with default prod value for minimumFetchIntervalMillis. --------- Co-authored-by: Athira M <[email protected]>
Hi @sanehab, We have updated the documentation to clarify the recommended prod value for minimumFetchIntervalMillis. We appreciate you bringing this to our attention and apologize for the delay in response. |
In the following screenshot documentation is using a small value for minimumFetchIntervalMillis without specifying that.

I have expected the value to be the default production value, or to at least have a comment that this value is a development specific value.
Reading the following text will make you even more sure that the value that was used was a production value
Anyway this is not related, but i also think that the throttle error can be more descriptive
In order to know what happened here i had to go back to documentation/source code.
The text was updated successfully, but these errors were encountered: