Skip to content

docs: Add stories using GraphQL - #105

Draft
bastilian wants to merge 1 commit into
mainfrom
stories_about_graphql
Draft

docs: Add stories using GraphQL#105
bastilian wants to merge 1 commit into
mainfrom
stories_about_graphql

Conversation

@bastilian

Copy link
Copy Markdown
Owner

No description provided.

@bastilian bastilian added the documentation Improvements or additions to documentation label Sep 24, 2025
@qltysh

qltysh Bot commented Sep 24, 2025

Copy link
Copy Markdown

❌ 3 blocking issues (3 total)

Tool Category Rule Count
eslint Lint Prefer a more specific type to Function 2
eslint Lint Missing JSDoc @returns description. 1

Comment thread src/support/api/backend/index.js Outdated
Comment thread src/support/api/backend/index.js Outdated
@bastilian
bastilian force-pushed the stories_about_graphql branch from b9d7412 to 7492f6f Compare May 5, 2026 22:54
Comment thread src/support/components/DarkMode.js Outdated
Comment thread src/utilities/hooks/useQueryWithUtilities/useQueryWithUtilities.js
Comment thread src/utilities/hooks/useQueryWithUtilities/useQueryWithUtilities.js Outdated
@bastilian
bastilian force-pushed the stories_about_graphql branch 2 times, most recently from 74c23ce to 8423146 Compare May 18, 2026 18:58
@qltysh

qltysh Bot commented May 18, 2026

Copy link
Copy Markdown

Qlty


Coverage Impact

⬇️ Merging this pull request will decrease total coverage on main by 0.25%.

Modified Files with Diff Coverage (2)

RatingFile% DiffUncovered Line #s
Coverage rating: F Coverage rating: F
...ties/hooks/useQueryWithUtilities/hooks/useQueryTotalBatched.js0.0%40-80
Coverage rating: F Coverage rating: F
...utilities/hooks/useQueryWithUtilities/useQueryWithUtilities.js0.0%59-91
Total0.0%
🤖 Increase coverage with AI coding...
In the `stories_about_graphql` branch, add test coverage for this new code:

- `src/utilities/hooks/useQueryWithUtilities/hooks/useQueryTotalBatched.js` -- Line 40-80
- `src/utilities/hooks/useQueryWithUtilities/useQueryWithUtilities.js` -- Line 59-91

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

* This hook is a wrapper around [TanStack's `useQuery`](https://tanstack.com/query/latest) and provides functions that can be directly utilised with a TableToolsTabls.
*
* @param {object} [options] Options
* @param {Function} [options.fetchFn] Fetch function that will be passed on to `useQuery`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Prefer a more specific type to Function [eslint:jsdoc/reject-function-type]

* @param {object|Array} [options.queue] A "queue" array or object of fetchFn that should be called instead of a fetchFn
* @param {object} [options.tableQueries] Options to pass to the `useTableQueries hook
* @param {object} [options.totalBatched] Options to pass to the `useQueryTotalBatched` hook
* @param {Function} [options.combineParamsWithTableState] Function to customise combining the table state with other parameters

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Prefer a more specific type to Function [eslint:jsdoc/reject-function-type]

* @param {object} [options.totalBatched] Options to pass to the `useQueryTotalBatched` hook
* @param {Function} [options.combineParamsWithTableState] Function to customise combining the table state with other parameters
*
* @returns {object}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing JSDoc @returns description. [eslint:jsdoc/require-returns-description]

@bastilian
bastilian force-pushed the stories_about_graphql branch from 8423146 to c9353c1 Compare May 19, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant