Skip to content

Conversation

victordidenko
Copy link

Hello!

Firebase JS SDK lacks realtime updates support.
I'm not sure why.

But I checked, how Android SDK is working, and implemented my own wrapper around @firebase/remote-config, which supports realtime updates. One thing I miss though — is remote config version extraction from the .../firebase:fetch endpoint response, in order to use it later in .../firebase:streamFetchInvalidations request payload.

I am patching local @firebase/remote-config package using pnpm patch, but I need to redo the patch every time new version came out.

So I decided it worth trying to make a PR. I figured, that if you are planning to add realtime updates support into JS SDK in future, you will have to add this change anyways.

It doesn't introduce any changes from regular user's perspective, not breaking, nor minor. So it could be new patch version. And it will help me a lot, so I could remove my local package patching.

Thank you for your attention!

@victordidenko victordidenko requested review from a team and erikeldridge as code owners July 21, 2025 11:18
Copy link

google-cla bot commented Jul 21, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link

changeset-bot bot commented Jul 21, 2025

🦋 Changeset detected

Latest commit: e30c884

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@firebase/remote-config Patch
firebase Patch
@firebase/remote-config-compat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@tusharkhandelwal8
Copy link

Hi @victordidenko, we are in the process of launching Realtime for Web support in the next few weeks. This PR(#9221) contains all the changes, so I believe we don't need to add this change and can close the PR for now. Thanks!

@victordidenko
Copy link
Author

That's great news!
Yes, as far as I can see, PR #9221 contains necessary changes indeed.
Thank you, I'm closing this then and will wait for the official Realtime for Web support 👍

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.

3 participants