Skip to content

Fix suspense #777

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 3 commits into from
Nov 17, 2020
Merged

Fix suspense #777

merged 3 commits into from
Nov 17, 2020

Conversation

promer94
Copy link
Collaborator

fix #494

@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 17, 2020

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 5755f67:

Sandbox Source
SWR-Basic Configuration
SWR-States Configuration
SWR-Infinite Configuration

@promer94
Copy link
Collaborator Author

Copy link
Member

@shuding shuding left a comment

Choose a reason for hiding this comment

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

Thanks for the fix! Can confirm that error is gone. (BTW I suggest renaming this ref to initialMountedRef, what do you think?)

@shuding shuding merged commit 7fe001c into vercel:master Nov 17, 2020
@huozhi
Copy link
Member

huozhi commented Nov 17, 2020

Thanks for the fix! Can confirm that error is gone. (BTW I suggest renaming this ref to initialMountedRef, what do you think?)

yup the first unmountRef is kinda confused, not really for mount/unmount, but for dependency changes. maybe we can rename it later as well

@promer94 promer94 deleted the fix-suspense branch November 17, 2020 09:08
@shuding
Copy link
Member

shuding commented Nov 17, 2020

@huozhi yep I agree 👍

shuding added a commit that referenced this pull request Dec 16, 2020
* 0.3.8

* replace rIC with rAF (#744)

* Fix race condition when calling mutate synchronously (#735)

* fix race condition when calling mutate synchronously

* fix test

* add comment

* fix code reviews

* refactor: support SSR in Deno (#754)

* refactor: support SSR in Deno

* refactor: improve Deno determining

* Add @ts-ignore

Co-authored-by: Shu Ding <[email protected]>

Co-authored-by: Shu Ding <[email protected]>

* fix eslint error (#768)

* Fix `mutateCallback` types (#745)

* Fix `mutateCallback` types

* WIP

* Add CodeSandbox CI (#769)

* add CodeSandbox CI

* add new line

* fix install cmd

Co-authored-by: Paco <[email protected]>

* dispatch's payload type is actionType and run lint (#772)

* chore: payload is actionType

* chore: move a ts-ignore comment

* Fix suspense (#777)

* fix #494

* add comment

* rename to initialMountedRef

* 0.3.9

* fix: mark isValidating as false when key is falsy (#757)

* fix: tear down when key turns to empty

* use false for empty key

* Fix README.md typo (#783)

'/api/data' => '/api/user' in "Multiple Arguments"

* fix: do mount check in config callback (#787)

* Update api-hooks example README.md (#790)

Updated the Vercel deploy link to the correct directory

* Return '@null' if args is null ASAP (#767)

* chore: return 'null' if arg[i] is null ASAP

* chore: update comment

* chore: use continue

* Bump ini from 1.3.5 to 1.3.8 (#806)

Bumps [ini](https://github.com/isaacs/ini) from 1.3.5 to 1.3.8.
- [Release notes](https://github.com/isaacs/ini/releases)
- [Commits](npm/ini@v1.3.5...v1.3.8)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* update test

Co-authored-by: Shu Ding <[email protected]>
Co-authored-by: X <[email protected]>
Co-authored-by: Umidbek Karimov <[email protected]>
Co-authored-by: Paco <[email protected]>
Co-authored-by: matamatanot <[email protected]>
Co-authored-by: Jiachi Liu <[email protected]>
Co-authored-by: sAy <[email protected]>
Co-authored-by: William Crutchfield <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

React experimental concurrent issue
3 participants