Skip to content

Adds a fairly complete Tanstack-query API using ky as a configurable transport.#72026

Merged
wilkie merged 13 commits into
stagingfrom
ngfp/tanstack-query-api
May 13, 2026
Merged

Adds a fairly complete Tanstack-query API using ky as a configurable transport.#72026
wilkie merged 13 commits into
stagingfrom
ngfp/tanstack-query-api

Conversation

@wilkie

@wilkie wilkie commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

This:

  • adds the @tanstack/react-query though using it and adding devtools is follow-up work
  • zod-powered schema-validation to the existing ky API layer
  • Extra transports for mocks and replay (indexed-db) for future offline dev-work
  • fully validated API calls and query hooks covering most of the level and user loading APIs

@wilkie wilkie force-pushed the ngfp/tanstack-query-api branch from 1f225c3 to 7024502 Compare April 9, 2026 00:11
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

🖼️ Storybook Visual Comparison Report

✅ No Storybook eyes differences detected!

Comment thread frontend/packages/core/src/api/QueryClientProvider.tsx Outdated
Comment thread frontend/packages/core/src/api/bootstrapApiClient.ts Outdated
Comment thread frontend/packages/core/src/api/dashboard/auth/auth.api.ts Outdated
Comment thread frontend/packages/core/src/api/dashboard/levels/levels.api.ts Outdated
Comment thread frontend/packages/core/src/api/dashboard/metrics/README.md
@wilkie wilkie marked this pull request as ready for review April 27, 2026 23:00
@wilkie wilkie marked this pull request as draft April 27, 2026 23:00
@wilkie wilkie force-pushed the ngfp/tanstack-query-api branch 2 times, most recently from 9430ae0 to aa51477 Compare April 28, 2026 04:34
@wilkie wilkie marked this pull request as ready for review April 28, 2026 16:15
@wilkie wilkie force-pushed the ngfp/tanstack-query-api branch from e8555fa to f0cd2ff Compare May 4, 2026 18:40
@wilkie wilkie force-pushed the ngfp/tanstack-query-api branch from f0cd2ff to 9ba5465 Compare May 13, 2026 18:41
@wilkie wilkie merged commit 05a8fc8 into staging May 13, 2026
14 checks passed
@wilkie wilkie deleted the ngfp/tanstack-query-api branch May 13, 2026 20:43
Hamms added a commit that referenced this pull request May 15, 2026
* Rename copilot-instructions.md to copilot.instructions.md

* refactor(i18n): improve locale handling (#72641)

* add indices (#72648)

* Update schema cache dump after schema changes.

* Restore Original Drone Cache (#72649)

* Restore Original Drone Cache

Follow-up to #68860,
which targeted a temporary cache file to minimize disruption.

* rebuild original cache image

* Revert "rebuild original cache image"

This reverts commit f323260.

* Font size updates for external file types (#72660)

* update stylesheet for external files

* add label and li

* script

* video tylsheet

* Revert "video tylsheet"

This reverts commit cbf9bf1.

* update external files

* update script

* delete script

* Video levels CSS change/markdown edit (#72662)

* Font size updates for free-response levels (#72667)

* Update application.scss

* remove updates from .text-match and .peer-review

* Update application.scss

* consolidate

* Update application.scss

* Adds a fairly complete Tanstack-query API using ky as a configurable transport. (#72026)

* Adds a fairly complete Tanstack-query API using ky as a configurable transport.

* Adds transports documentation.

* Commits yarn.lock.

* Type fixes for music lab

* Updates music-lab tests to wrap the mock transport.

* Update frontend/packages/core/src/api/QueryClientProvider.tsx

Co-authored-by: Stephen Liang <stephenliang@users.noreply.github.com>

* Update frontend/packages/core/src/api/bootstrapApiClient.ts

Co-authored-by: Stephen Liang <stephenliang@users.noreply.github.com>

* Apply suggestion from @wilkie

* Refactors the level properties request params and API calls.

* Determines level properties URL with type discrimination.

* Places @tanstack/react-query into catalog.

* Fixes type issue in design-system-storybook

* Fixes format issue.

---------

Co-authored-by: Stephen Liang <538214+stephenliang@users.noreply.github.com>
Co-authored-by: Stephen Liang <stephenliang@users.noreply.github.com>

* levelbuilder content changes (-robo-commit) (#72682)

Co-authored-by: Continuous Integration <dev@code.org>
Co-authored-by: Stephen Liang <538214+stephenliang@users.noreply.github.com>

* Update header font sizes for free-response questions (#72684)

* update header sizes

* revert header size changes for video and external

* remove header styling for free-response

* manually resolve merge conflict with levelbuilder

---------

Co-authored-by: Hannah Bergam <hannahbergam@gmail.com>
Co-authored-by: Artem Vavilov <artem.vavilov.7@gmail.com>
Co-authored-by: Code.org Deploy Systems <dev@code.org>
Co-authored-by: Mark Barnes <mark.barnes@code.org>
Co-authored-by: fisher-alice <107423305+fisher-alice@users.noreply.github.com>
Co-authored-by: Eric Aderhold <eric.aderhold@code.org>
Co-authored-by: alex-m-brown <alex.brown@code.org>
Co-authored-by: wilkie <david.wilkinson@code.org>
Co-authored-by: Stephen Liang <538214+stephenliang@users.noreply.github.com>
Co-authored-by: Stephen Liang <stephenliang@users.noreply.github.com>
Co-authored-by: Alice Fisher <alice.fisher@code.org>
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.

2 participants