Skip to content

Disable Fetch for ReactNative #5643

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

Merged
merged 5 commits into from
Oct 20, 2021
Merged

Disable Fetch for ReactNative #5643

merged 5 commits into from
Oct 20, 2021

Conversation

schmidt-sebastian
Copy link
Contributor

@schmidt-sebastian schmidt-sebastian commented Oct 20, 2021

We have to use XHR instead of Fetch for ReactNative since response.blob is not supported (see facebook/react-native#27741)

Addresses #1674

@changeset-bot
Copy link

changeset-bot bot commented Oct 20, 2021

🦋 Changeset detected

Latest commit: d1d3f28

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

This PR includes changesets to release 3 packages
Name Type
@firebase/firestore Patch
firebase Patch
@firebase/firestore-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

@google-cla google-cla bot added the cla: yes label Oct 20, 2021
@@ -20,7 +20,7 @@
"dev": "rollup -c -w",
"lint": "eslint -c .eslintrc.js '**/*.ts' --ignore-path '../../.gitignore'",
"lint:fix": "eslint --fix -c .eslintrc.js '**/*.ts' --ignore-path '../../.gitignore'",
"prettier": "prettier --write '*.js' '*.ts' '@(lite|src|test)/**/*.ts'",
"prettier": "prettier --write '*.js' '@(lite|src|test)/**/*.ts'",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prettier fails since there are no *.ts files at the root anymore.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Oct 20, 2021

Binary Size Report

Affected SDKs

  • @firebase/firestore

    Type Base (fc9ec0f) Head (2547e15) Diff
    browser 225 kB 225 kB +12 B (+0.0%)
    esm5 282 kB 282 kB +21 B (+0.0%)
    main 423 kB 423 kB +32 B (+0.0%)
    module 225 kB 225 kB +12 B (+0.0%)
    react-native 225 kB 226 kB +12 B (+0.0%)
  • firebase

    Type Base (fc9ec0f) Head (2547e15) Diff
    firebase-compat.js 749 kB 749 kB +9 B (+0.0%)
    firebase-firestore-compat.js 277 kB 277 kB +6 B (+0.0%)
    firebase-firestore.js 763 kB 763 kB +57 B (+0.0%)

Test Logs

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Oct 20, 2021

Size Analysis Report

Affected Products

Diffs between base commit (fc9ec0f) and head commit (b6e9b6a) are too large (1,122,347 characters) to display.

Please check below links to see details from the original test log.

@johnnyoshika
Copy link

We have to use Fetch instead of XHR for ReactNative since response.blob is not supported (see facebook/react-native#27741)

Addresses #1674

Is the comment incorrect? Should it say use XHR instead of Fetch for ReactNative?

@schmidt-sebastian
Copy link
Contributor Author

Yeah, it should be the other way around. Let me fix the discription.

@firebase firebase locked and limited conversation to collaborators Nov 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants