Skip to content

docs(useQueryWithUtilities): Add story with non default request result#111

Merged
bastilian merged 1 commit intomainfrom
non_default_request_results
Oct 6, 2025
Merged

docs(useQueryWithUtilities): Add story with non default request result#111
bastilian merged 1 commit intomainfrom
non_default_request_results

Conversation

@bastilian
Copy link
Owner

@bastilian bastilian commented Sep 29, 2025 •
edited
Loading

This adds an example for the useQueryWithUtilities with a request that returns a non-default response. The example response in this case has a data prop, but is not an array, which has caused issues with some API responses in apps.

How to test:

  1. Run Story book
  2. Verify there are no regressions in stories
  3. Verify the example/story is sane

@bastilian bastilian self-assigned this Sep 29, 2025
@bastilian bastilian added the utilities For things that are part of the utilities label Sep 29, 2025
@bastilian bastilian force-pushed the non_default_request_results branch from 8657005 to f6ae512 Compare September 29, 2025 22:04
@qltysh
Copy link

qltysh bot commented Sep 29, 2025 •
edited
Loading

Diff Coverage: The code coverage on the diff in this pull request is 0.0%.

Total Coverage: This PR will not change total coverage.

🛟 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.

@bastilian bastilian force-pushed the non_default_request_results branch from f6ae512 to 57cc840 Compare October 3, 2025 18:20
@bastilian bastilian marked this pull request as ready for review October 3, 2025 18:25
@bastilian bastilian added the rebase PRs with this label should be "Rebased and merge(d)", rather than "Squash and merge(d)". label Oct 3, 2025
LightOfHeaven1994
LightOfHeaven1994 approved these changes Oct 6, 2025
View reviewed changes
Copy link
Collaborator

@LightOfHeaven1994 LightOfHeaven1994 left a comment

Choose a reason for hiding this comment

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

LGTM, works as expected! Thank you ❤️

@bastilian bastilian merged commit 33e2cc9 into main Oct 6, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LightOfHeaven1994 LightOfHeaven1994 LightOfHeaven1994 approved these changes

Assignees

@bastilian bastilian

Labels

rebase PRs with this label should be "Rebased and merge(d)", rather than "Squash and merge(d)". utilities For things that are part of the utilities

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bastilian @LightOfHeaven1994