Package Scope

• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

@suspensive/swr

• Add to an existing package
• New package

Package name: swr

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

all

Package name: all

Describe the bug

Package Scope

Package name:

Describe the bug

Expected behavior

To Reproduce

Possible Solution

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

docs

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

useSuspenseQuery

Package name:
@suspensive/react-query

Describe the bug

I'm trying to update state from onSuccess callback of the query this way

const Test = () => {
  const [test, setTest] = useState(0);

  useSuspensiveQuery({
    queryKey: ['some-key'],
    queryFn: somePromise,
    onSuccess: () => {
      setTest(1)
    }
  })

  console.log(test); // it would be 0
  
}

Expected behavior

State should be 1 on success.

To Reproduce

See below

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Oh Thanks, you're right. different of isDifferentArray is limited different. I didn't think about this when I created this function. I made it by referring to the comparison operation of dependencyList of useEffect hook.
Isn't there a better name for this function? If there is a better name, I think it would be nice to make it an issue and work on it in the next PR.

It would be nice to change function name, too!
I think we should think of a good name!

Originally posted by @ChanhyukPark-Tech in #28 (comment)

Package Scope

docs

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name: suspensive/react

Overview

The test coverage for the AsyncBoundary component is currently insufficient. The given test case performs a simple rendering test, but there are no tests that verify whether the key features of AsyncBoundary work properly. Given that AsyncBoundary is a critical component that manages async loading states and error handling, it's essential that comprehensive tests are in place.

Describe the solution you'd like

I propose adding tests for the async loading and error handling aspects of the AsyncBoundary component.

Additional context

AsyncBoundary test code

// suspensive test code
import { render } from '@testing-library/react'

describe('test', () => {
  it('should test', () => {
    const TEST_TEXT = 'suspensive'
    const fixture = render(<div>{TEST_TEXT}</div>)

    expect(fixture.getByText(TEST_TEXT)).toBeInTheDocument()
  })
})

At present, our test code seems to exist only to fulfill the requirement of having test code. What we need is true testing that effectively validates our application's behavior. This will be instrumental in facilitating the next leap forward for our Suspensive service...

          > currently, we use stringified key yet, but next Pull Request, I want to change it suspenseCache's methods need to work with index of key with hierarchy like queryKey of @tanstack/query

I think this is need to be implemented in this Pull Request. so I change this Pull Request as draft. after that, I'll request review

Originally posted by @manudeli in #83 (comment)

Package Scope

all

• Add to an existing package

Package name: all

Overview

Describe the solution you'd like

TanStack/query#4738

Additional context

Package Scope

• Add to an existing package

Package name: all

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name: @suspensive/react

Overview

Describe the solution you'd like

Additional context

Add docusaurus i18n

https://docusaurus.io/docs/i18n/introduction

• Add to an existing package
• New package

Package name: all

Overview

Describe the solution you'd like

Additional context

Package Scope

react

• Add to an existing package
• New package

Package name: react

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name: react

Overview

Describe the solution you'd like

Additional context

Package Scope

Package name:

Describe the bug

All the Edit this page links on every page of the official documentation are incorrect.

For example, the Edit this page link on Motivation page looks like this:
https://github.com/suspensive/react/docs/intro/motivation.i18n.md

The Edit this page links in the pages below all come out as 'Not Found'.

Is document i18n still WIP?

Expected behavior

To Reproduce

Possible Solution

Additional context

Package Scope

• Add to an existing package
• New package

Package name: @suspensive/react

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name: @suspensive/react-query

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name: entire package, root typescript version

Overview

Type script version 5 was actually released, so it would be good to apply it.

Describe the solution you'd like

Upgrade entire typescript version

Additional context

Package Scope

Package name:
suspensive/react

Describe the bug

broken query triggers boundary but it does not refetch it on the next mount.

Expected behavior

Query should be refetched on the second mount of the component

To Reproduce

https://github.com/timsofteng/query-boundary-bug

suspensive branch

steps:

yarn install && yarn dev
Open http://127.0.0.1:5173/one
Click to page two. See an error in query and in boundary.
Click to page one. Page one should be rendered ok.
Click to page two. Boundary is displayed but query does not refetch.

Possible Solution

Additional context

Package Scope

all

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

All

• Add to an existing package
• New package

Package name: all

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name: @suspensive/react

Overview

• Delay
• SuspensiveProvider, SuspensiveConfigs
• suspenseCache, useSuspenseCache

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name: @suspensive/react

Overview

Describe the solution you'd like

Additional context

Package Scope

useSuspensiveQuery

Package name:
@suspensive/react-query

Describe the bug

If useSuspensiveQuery has enabled: boolean param then it means data may be undefined (DataType: TData | undefined).
Howerver it is defined despite of enabled param. It may breaks component if user try to get some inner data like data.someInnerKey.
Uncaught TypeError: Cannot read properties of undefined

Expected behavior

I guess the lib should mark data type as TData | undefined and status "loading" | "success" in case useSuspensiveQuery has enabled param.

To Reproduce

Possible Solution

Additional context

Package Scope

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Relevant code or config

fallback: ReactNode | ((props: { error: Error; reset: (...args: unknown[]) => void }) => ReactNode)

Problem description:

fallback type has reset prop and it take arg as a factor. but It's not used anywhere

Suggested solution:

i think it works too

fallback: ReactNode | ((props: { error: Error; reset: () => void }) => ReactNode)

Package Scope

• Add to an existing package

Package name: @suspensive/react

Overview

reactwg/react-18#86

Implement fixing with compatiable react 17 or under with this library
https://www.npmjs.com/package/use-sync-external-store

Reference

react-redux's solution https://github.com/reduxjs/react-redux/blob/master/package.json#L77

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name: docs

Overview

Describe the solution you'd like

Additional context

https://github.com/manudeli/gof-design-patterns

Package Scope

• Add to an existing package
• New package

Package name:

Overview

I'm actually using suspensive well because you provided me a good library. I'm looking forward to it more and more because you're continuing to update new features.
As a library user, I wonder what features have been added depending on the version. It would be nice if we could check the update log and so on somewhere other than PR or commit!

Describe the solution you'd like

• github release ?

Additional context

Package Scope

https://www.youngs-log.kr/react-query/query-key-structuring

• Add to an existing package
• New package

Package name:

Overview

Describe the solution you'd like

Additional context

Package Scope

All

• Add to an existing package
• New package

Package name: all

Overview

Describe the solution you'd like

Additional context

Resolve TanStack/query#5303

• Add to an existing package

Package name: @suspensive/react-query

Package Scope

docs

• Add to an existing package
• New package

Package name: docs

Overview

Describe the solution you'd like

Additional context

Package Scope

• Add to an existing package

Package name: @suspensive/react-query