Skip to content
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

fix(persist): remove environment check #3658

Merged
merged 1 commit into from May 30, 2022

Conversation

TkDodo
Copy link
Collaborator

@TkDodo TkDodo commented May 30, 2022

because we can now pass in undefined for persisters during SSR, and the persisters themselves are not window related anymore, as we don't use window.localStorage directly

because we can now pass in undefined for persisters during SSR, and the persisters themselves are not window related anymore, as we don't use window.localStorage directly
@vercel
Copy link

vercel bot commented May 30, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
react-query ✅ Ready (Inspect) Visit Preview May 30, 2022 at 7:32AM (UTC)

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8141a29:

Sandbox Source
tannerlinsley/react-query: basic Configuration
tannerlinsley/react-query: basic-typescript Configuration

@codecov
Copy link

codecov bot commented May 30, 2022

Codecov Report

Merging #3658 (8141a29) into beta (0c04e7d) will decrease coverage by 0.00%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##             beta    #3658      +/-   ##
==========================================
- Coverage   97.11%   97.11%   -0.01%     
==========================================
  Files          50       50              
  Lines        2393     2389       -4     
  Branches      708      705       -3     
==========================================
- Hits         2324     2320       -4     
  Misses         67       67              
  Partials        2        2              
Impacted Files Coverage Δ
src/createWebStoragePersister/index.ts 86.48% <ø> (ø)
src/persistQueryClient/persist.ts 95.00% <94.44%> (-0.46%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c04e7d...8141a29. Read the comment docs.

@TkDodo TkDodo merged commit 8095859 into TanStack:beta May 30, 2022
@TkDodo TkDodo deleted the feature/remove-persist-env-check branch May 30, 2022 07:34
@tannerlinsley
Copy link
Collaborator

🎉 This PR is included in version 4.0.0-beta.19 🎉

The release is available on:

Your semantic-release bot 📦🚀

GLabat pushed a commit to GLabat/react-query that referenced this pull request Jun 27, 2022
…k-beta

* 'beta' of https://github.com/TanStack/query: (140 commits)
  Fix Best of JS badge in README (TanStack#3695)
  Update config.json
  docs: adapter teasers
  docs: rewrite typescript docs
  tests: stabilize
  docs: Update graphql.md to fix link (TanStack#3683)
  docs: fix link typo in README.md (TanStack#3679)
  Update README.md
  update readme
  Update config.json
  Update index.js
  examples: vendor to platform directories
  docs: migrate to tanstack.com
  fix(persist): introduce in-house synchronous Storage interface (TanStack#3673)
  docs: restructure migration guide by importance of breaking changes
  fix: `replaceDeepEqual` special case for non-plain arrays (TanStack#3669)
  feat: make InfiniteQueryObserver's type more robust (TanStack#3571)
  fix(types): remove non-void constraint from queryFn result (TanStack#3666)
  fix(persist): remove environment check (TanStack#3658)
  fix(types): make sure queryClient.setQueriesData can return undefined from the updater (TanStack#3657)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants