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(react-query): update JSDoc for useSuspenseQueries, useSuspenseInfiniteQuery #923

Merged
merged 3 commits into from
Jun 14, 2024

Conversation

gwansikk
Copy link
Collaborator

Overview

Updates the JSDoc for the useSuspenseQueries and useSuspenseInfiniteQuery functions.

PR Checklist

  • I did below actions if need
  1. I read the Contributing Guide
  2. I added documents and tests.

Copy link

changeset-bot bot commented Jun 14, 2024

🦋 Changeset detected

Latest commit: bce9865

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

This PR includes changesets to release 2 packages
Name Type
@suspensive/react-query Patch
@suspensive/react 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

Copy link

vercel bot commented Jun 14, 2024

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

Name Status Preview Comments Updated (UTC)
suspensive.org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 14, 2024 0:03am
v1.suspensive.org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 14, 2024 0:03am
visualization.suspensive.org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 14, 2024 0:03am

@@ -27,7 +27,7 @@ export interface UseSuspenseInfiniteQueryOptions<
> {}

/**
* This hook is wrapping useInfiniteQuery of @tanstack/react-query v4 with default suspense option.
* This hook is wrapping useInfiniteQuery of `@tanstack/react-query` v4 with default suspense option.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I've made revisions for consistency.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks! So much better jsdoc 👍

AS-IS

image

TO BE

image

@@ -109,6 +109,10 @@ export type SuspenseQueriesResults<
: // Fallback
Array<UseSuspenseQueryResult>

/**
* This hook is wrapping useQueries of `@tanstack/react-query` v4 with default suspense option.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I added the JSDoc for this function as it was missing.

Copy link
Member

Choose a reason for hiding this comment

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

👍

@codecov-commenter
Copy link

codecov-commenter commented Jun 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.44%. Comparing base (26013ac) to head (bce9865).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #923   +/-   ##
=======================================
  Coverage   83.44%   83.44%           
=======================================
  Files          33       33           
  Lines         435      435           
  Branches       99      100    +1     
=======================================
  Hits          363      363           
  Misses         62       62           
  Partials       10       10           
Components Coverage Δ
@suspensive/react 96.98% <ø> (ø)
@suspensive/react-query 13.79% <ø> (ø)
@suspensive/react-await 100.00% <ø> (ø)
@suspensive/react-image 23.52% <ø> (ø)

@manudeli
Copy link
Member

Thanks! You made so much better jsdoc!

@manudeli manudeli merged commit 1e7fbbe into toss:main Jun 14, 2024
12 checks passed
@github-actions github-actions bot mentioned this pull request Jun 14, 2024
manudeli pushed a commit that referenced this pull request Jun 14, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @suspensive/react-query@2.0.6

### Patch Changes

- [#923](#923)
[`1e7fbbe`](1e7fbbe)
Thanks [@gwansikk](https://github.com/gwansikk)! - fix(react-query):
update JSDoc for `useSuspenseQueries`, `useSuspenseInfiniteQuery`

-   Updated dependencies \[]:
    -   @suspensive/react@2.0.6

## @suspensive/react@2.0.6

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@gwansikk gwansikk deleted the jsdoc branch June 14, 2024 12:24
manudeli added a commit that referenced this pull request Aug 3, 2024
…seInfiniteQuery` (#923)

# Overview

Updates the JSDoc for the `useSuspenseQueries` and
`useSuspenseInfiniteQuery` functions.

## PR Checklist

- [x] I did below actions if need

1. I read the [Contributing
Guide](https://github.com/toss/suspensive/blob/main/CONTRIBUTING.md)
2. I added documents and tests.

---------

Co-authored-by: Jonghyeon Ko <jonghyeon@toss.im>
manudeli added a commit that referenced this pull request Aug 3, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @suspensive/react-query@2.0.6

### Patch Changes

- [#923](#923)
[`1e7fbbe`](1e7fbbe)
Thanks [@gwansikk](https://github.com/gwansikk)! - fix(react-query):
update JSDoc for `useSuspenseQueries`, `useSuspenseInfiniteQuery`

-   Updated dependencies \[]:
    -   @suspensive/react@2.0.6

## @suspensive/react@2.0.6

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
manudeli pushed a commit that referenced this pull request Aug 3, 2024
…seInfiniteQuery` (#923)

# Overview

Updates the JSDoc for the `useSuspenseQueries` and
`useSuspenseInfiniteQuery` functions.

## PR Checklist

- [x] I did below actions if need

1. I read the [Contributing
Guide](https://github.com/toss/suspensive/blob/main/CONTRIBUTING.md)
2. I added documents and tests.

---------
manudeli added a commit that referenced this pull request Aug 3, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @suspensive/react-query@2.0.6

### Patch Changes

- [#923](#923)
[`1e7fbbe`](1e7fbbe)
Thanks [@gwansikk](https://github.com/gwansikk)! - fix(react-query):
update JSDoc for `useSuspenseQueries`, `useSuspenseInfiniteQuery`

-   Updated dependencies \[]:
    -   @suspensive/react@2.0.6

## @suspensive/react@2.0.6
manudeli added a commit that referenced this pull request Aug 3, 2024
…seInfiniteQuery` (#923)

# Overview

Updates the JSDoc for the `useSuspenseQueries` and
`useSuspenseInfiniteQuery` functions.

## PR Checklist

- [x] I did below actions if need

1. I read the [Contributing
Guide](https://github.com/toss/suspensive/blob/main/CONTRIBUTING.md)
2. I added documents and tests.

---------

Co-authored-by: Jonghyeon Ko <jonghyeon@toss.im>
manudeli added a commit that referenced this pull request Aug 3, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @suspensive/react-query@2.0.6

### Patch Changes

- [#923](#923)
[`1e7fbbe`](1e7fbbe)
Thanks [@gwansikk](https://github.com/gwansikk)! - fix(react-query):
update JSDoc for `useSuspenseQueries`, `useSuspenseInfiniteQuery`

-   Updated dependencies \[]:
    -   @suspensive/react@2.0.6

## @suspensive/react@2.0.6
manudeli added a commit that referenced this pull request Aug 3, 2024
…seInfiniteQuery` (#923)

# Overview

Updates the JSDoc for the `useSuspenseQueries` and
`useSuspenseInfiniteQuery` functions.

## PR Checklist

- [x] I did below actions if need

1. I read the [Contributing
Guide](https://github.com/toss/suspensive/blob/main/CONTRIBUTING.md)
2. I added documents and tests.

---------

Co-authored-by: Jonghyeon Ko <jonghyeon@toss.im>
manudeli added a commit that referenced this pull request Aug 3, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @suspensive/react-query@2.0.6

### Patch Changes

- [#923](#923)
[`1e7fbbe`](1e7fbbe)
Thanks [@gwansikk](https://github.com/gwansikk)! - fix(react-query):
update JSDoc for `useSuspenseQueries`, `useSuspenseInfiniteQuery`

-   Updated dependencies \[]:
    -   @suspensive/react@2.0.6

## @suspensive/react@2.0.6
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.

3 participants