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

chore: remove useless overload #1993

Merged
merged 1 commit into from
May 25, 2022
Merged

chore: remove useless overload #1993

merged 1 commit into from
May 25, 2022

Conversation

promer94
Copy link
Collaborator

No description provided.

@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 dd90fe6:

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

@huozhi huozhi merged commit def1367 into vercel:mutate May 25, 2022
@huozhi
Copy link
Member

huozhi commented May 25, 2022

👍

huozhi pushed a commit that referenced this pull request Jun 6, 2022
huozhi pushed a commit that referenced this pull request Jun 25, 2022
shuding added a commit that referenced this pull request Jun 26, 2022
* feat: mutate filter

* tweak typing

* add test

* tweak typings

* chore: remove useless overload (#1993)

* fix typing

* chore: change internalMutate type (#1997)

* chore: change internalMutate type

* chore: useoverload for internal mutate

* store keys in global state

* add type truthy key

* polish truthy key

* dont remove key

* add clear test case, fix empty key

* store key in cache

* use values

* fix cache test

* pass original argument to the filter

* fix types

* fix types

* fix failed tests; use useSyncExternalStore and memorized selector

* move function declaration

* fix infinite use cases

* fix lint error

Co-authored-by: Yixuan Xu <yixuanxu94@outlook.com>
Co-authored-by: Shu Ding <g@shud.in>
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.

None yet

3 participants