Skip to content

docs: add a TanStack Query value card to the home page#136

Merged
justin808 merged 2 commits into
mainfrom
jg/docs-tanstack-query-blurb
Jun 28, 2026
Merged

docs: add a TanStack Query value card to the home page#136
justin808 merged 2 commits into
mainfrom
jg/docs-tanstack-query-blurb

Conversation

@justin808

@justin808 justin808 commented Jun 28, 2026

Copy link
Copy Markdown
Member

Adds a Modern data fetching value card to the home page mentioning TanStack Query (client-side caching, mutations, and first-paint data on a Rails JSON API, without a second backend).

Scope note: the examples page already lists the TanStack Starter card (demos.ts), and the Using TanStack Query guide auto-syncs into /docs at build — so the blurb is the only content addition.

Second commit (CI fix): refreshes the generated prompt artifacts (prompts.ts / prompts.json / prompts/llms.txt) from upstream prompts.yml. The prepare:prompts guard was failing on every PR because those committed artifacts lagged react_on_rails main (shakacode/react_on_rails#4232). Verified locally: npm run build (prepare:prompts:check + docusaurus build) passes.

Closes #135

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated app-start guidance to clarify the default setup, licensing requirements across environments, and when to use the standard open-source scaffold.
  • New Features
    • Added a new homepage highlight about modern data fetching, showing how Rails JSON APIs can pair with TanStack Query.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5ab73883-1fca-4d1c-b338-a9f93a0968f6

📥 Commits

Reviewing files that changed from the base of the PR and between 6d2329e and 189d50d.

📒 Files selected for processing (4)
  • prototypes/docusaurus/src/constants/prompts.ts
  • prototypes/docusaurus/src/pages/index.tsx
  • prototypes/docusaurus/static/prompts.json
  • prototypes/docusaurus/static/prompts/llms.txt

Walkthrough

The create-app prompt string is updated across three files (prompts.ts, prompts.json, llms.txt) to add explicit Pro-path, licensing/token, and --standard guidance. A new "Modern data fetching" card is added to the homepage valueCards array.

Changes

Content updates

Layer / File(s) Summary
create-app prompt update
prototypes/docusaurus/src/constants/prompts.ts, prototypes/docusaurus/static/prompts.json, prototypes/docusaurus/static/prompts/llms.txt
Replaces the prior general prompt with explicit Pro-path defaults, per-environment token/license requirements, and --standard usage guidance across all three prompt source files.
Homepage value card
prototypes/docusaurus/src/pages/index.tsx
Adds a "Modern data fetching" entry to the valueCards array describing TanStack Query pairing with Rails JSON APIs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

  • #135: This PR directly implements item 2 from the issue (adding a "Modern data fetching" blurb to index.tsx).

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning Prompt artifacts were modified even though the linked issue only calls for site marketing/content changes. Split the prompt regeneration into a separate PR or revert it from this changeset so the PR stays focused on the TanStack Query site updates.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly states the main change: adding a TanStack Query card to the home page.
Linked Issues check ✅ Passed The home-page TanStack Query blurb was added, and the PR context says the starter card already exists, satisfying the linked site-surface goals.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jg/docs-tanstack-query-blurb

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Jun 28, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds a new home-page value card for TanStack Query. The main change is:

  • A “Modern data fetching” card in the Docusaurus home page value-card list.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
prototypes/docusaurus/src/pages/index.tsx Adds one static TanStack Query value card to the home page card data.

Reviews (1): Last reviewed commit: "docs: add a TanStack Query value card to..." | Re-trigger Greptile

CI's prepare:prompts guard was red on every reactonrails.com PR because the
committed prompt artifacts lagged react_on_rails main (prompts.yml updated in
shakacode/react_on_rails#4232). Regenerated with `npm run prepare:prompts`
against main; verified `npm run build` passes (prepare:prompts:check + docusaurus build).
@github-actions

Copy link
Copy Markdown

Cloudflare preview deployed.

@justin808 justin808 merged commit e0461a4 into main Jun 28, 2026
3 checks passed
@justin808 justin808 deleted the jg/docs-tanstack-query-blurb branch June 28, 2026 01:12
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.

Site: surface TanStack Query support (examples card + feature blurb)

1 participant