-
Notifications
You must be signed in to change notification settings - Fork 928
React Native build breaks Writes #3243
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
Thanks for reporting the issue. I couldn't reproduce it running the code snippet you provided, but I also couldn't replicate your whole environment (I'm getting errors trying to run the |
I used expo init to create my project. I am not running the web-script since I also get errors using that. But I intended my project to be native only anyway. I am using the expo app on iPhone 11 pro simulator, software version: 13.5 & the expo app on realme X2 android version 10. On both devices I get the same warning and also the same debug from firestore, after enabling debug logging. Resulting Logsthis has been edited, I forgot to turn on the function that actually caused the error..
ReproI'm not sure if this is what you mean with repro project nor if you can view my project, I'm new to using GitHub. |
I would also like to mention, I used the same approach on a different app I was making a few months ago, in this app everything was working perfectly with firestore. But it broke after updating the firebase sdk. Now that I worked on this new app I pretty much re used the previous approach. I just tested the latest firebase sdk 7.15.3 in this app. And I get the same warning message. Also I am not able to add data anymore to my firestore. I was able to receive data from the firestore. Here is the log from where everything is still working fine
Here is the log of an action that tries to add data to firestore
|
Thanks for sending over the logs, which showcase the problem:
The In the meantime, you can "opt out" of this new code by installing an Base64 Polyfill like suggested here: https://stackoverflow.com/questions/60361519/cant-find-a-variable-atob |
Did installing a Base64 Polyfill fix the issue for you in the standalone test? Because for me the stackoverflow suggestion isn't working |
In the standalone test that just does a base64 encode/decode roundtrip, the encoding works if there is an |
Do you know when this will be fixed? The Polyfill suggestion with 7.15.0 is not working for me. Do you happen to know if there is a recent version I can use without this error? @schmidt-sebastian can you describe how you'd do an "atob/btoa fallback"? (This same problem is on stack overflow: https://stackoverflow.com/questions/62630423/firebase-react-native-invalid-stream-token) |
As of 7.15.4, we have reverted back to the previous behavior. You can find a guide to install the Polyfill here: https://stackoverflow.com/questions/60361519/cant-find-a-variable-atob/60369040#60369040 In the next release (fingers crossed), the Polyfill will no longer be needed. |
I just wanna say, with the new update 7.15.5 this issue was resolved. thank you! 👍 |
I am still kind of embarrassed by how long this took, but thanks for letting me know that I can now move on to other things :) |
#2929 # [REQUIRED] Describe your environment
Describe problem:
Everywhere I call firebase.firestore.collection('...').add({...});
I get a warning message:
Steps/Code to reproduce:
This will also happen if I import firestore elsewhere and call: firestore.collection(...).add({...});
package.json
The text was updated successfully, but these errors were encountered: