Skip to content

Refresh questions table when a question is marked as resolved - #19475

Closed
brittanyjoiner15 wants to merge 2 commits into
masterfrom
posthog/refresh-questions-table-on-resolve
Closed

Refresh questions table when a question is marked as resolved#19475
brittanyjoiner15 wants to merge 2 commits into
masterfrom
posthog/refresh-questions-table-on-resolve

Conversation

@brittanyjoiner15

Copy link
Copy Markdown
Contributor

Changes

Marking a question as resolved only revalidated the single-question SWR cache, so a questions table mounted elsewhere on the page kept showing the old status until a manual page refresh.

handleResolve in useQuestion now revalidates every cached /api/questions request — the question itself plus any list/table queries from useQuestions — using SWR's global mutate with a key matcher.

Why: reported in Slack — after marking a question resolved on /questions, the check mark didn't appear in the table without refreshing the page.

Checklist

  • I've read the docs and/or content style guides.
  • Words are spelled using American English
  • Use relative URLs for internal links
  • I've checked the pages added or changed in the Vercel preview build
  • If I moved a page, I added a redirect in vercel.json

Created with PostHog from a Slack thread

Marking a question as resolved only mutated the single-question SWR cache, so a questions table mounted elsewhere kept showing stale status until the page was refreshed.

handleResolve now revalidates every cached /api/questions request (the question itself plus any list/table queries) via SWR's global mutate.

Generated-By: PostHog Desktop
Task-Id: 68c5219f-0202-41d2-837d-ef7271bd72a5
@github-actions github-actions Bot added the website About the website (beyond just landing pages) label Aug 13, 2026
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Deploy preview

Status Details Updated (UTC)
🟢 Ready View preview Aug 13, 2026 09:57PM

@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Bundle report

Total JS (gzip)

7.55 MiB (+0.1 KiB / +0.0%)

Eager graph (modules shipped in each entrypoint's initial chunks)

Entrypoint Eager size Budget Modules
app 16.90 MiB (+0.6 KiB / +0.0%) report-only 2018
Largest modules in the app closure
Module Size
./src/data/mcp-tools.json 990.6 KiB
css ./node_modules/.pnpm/css-loader@5.2.7_webpack@5.101.3/node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[1]!./node_modules/.pnpm/postcss-loader@4.3.0_postcss@8.5.6_webpack@5.101.3/node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[2]!./src/styles/global.css 746.7 KiB
./src/components/Stickers/Stickers.tsx 696.4 KiB
./node_modules/.pnpm/@radix-ui+react-icons@1.3.2_react@18.3.1/node_modules/@radix-ui/react-icons/dist/react-icons.esm.js 481.4 KiB
./node_modules/.pnpm/rehype-raw@7.0.0/node_modules/rehype-raw/lib/index.js + 29 modules 395.1 KiB
./node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.cjs.js 364.8 KiB
./node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js 354.8 KiB
./src/hooks/useCustomers.tsx + 54 modules 354.5 KiB
./node_modules/.pnpm/react-markdown@8.0.7_@types+react@16.14.66_react@18.3.1/node_modules/react-markdown/lib/react-markdown.js + 88 modules 351.4 KiB
./src/components/ProductComparisonTable/index.tsx + 126 modules 301.4 KiB
./node_modules/.pnpm/cloudinary-core@2.14.0_lodash@4.17.21/node_modules/cloudinary-core/cloudinary-core.js 281.9 KiB
./src/components/SearchUI/index.tsx + 87 modules 273.0 KiB
./node_modules/.pnpm/@posthog+brand@0.8.0_react@18.3.1/node_modules/@posthog/brand/dist/generated/hoggies/svg/magnifying-glass.mjs 254.7 KiB
./node_modules/.pnpm/framer-motion@10.18.0_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/framer-motion/dist/es/render/dom/motion.mjs + 109 modules 253.9 KiB
./node_modules/.pnpm/d3@7.9.0/node_modules/d3/src/index.js + 208 modules 247.4 KiB

Eager-graph budgets are report-only until a baseline is established. Sizes are gzip of public/**/*.js; eager size is webpack module source bytes for the modules actually shipped in the entrypoint's initial chunks (post-tree-shake).

@brittanyjoiner15

Copy link
Copy Markdown
Contributor Author

Running locally to test this

@brittanyjoiner15
brittanyjoiner15 marked this pull request as ready for review August 13, 2026 21:35
@brittanyjoiner15

Copy link
Copy Markdown
Contributor Author

Wasn't working at first but worked with claude to get some changes and tested locally and seems to be working now.

@smallbrownbike

Copy link
Copy Markdown
Collaborator

Closed in favor of #19508

@github-project-automation github-project-automation Bot moved this from Backlog to Done in PostHog.com planning Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-forum website About the website (beyond just landing pages)

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants