Skip to content

Fix report output actions, canonical navigation, and SSR-safe overview charts#942

Closed
anth-volk wants to merge 2 commits intomainfrom
fix/fix-share-buttons
Closed

Fix report output actions, canonical navigation, and SSR-safe overview charts#942
anth-volk wants to merge 2 commits intomainfrom
fix/fix-share-buttons

Conversation

@anth-volk
Copy link
Copy Markdown
Collaborator

Fixes #941

Summary

  • expose the safe report-output header actions in shared mode while keeping shared setup read-only
  • switch setup and reproduce navigation to canonical owned/shared URLs and standardize the Back to report output breadcrumb
  • remove unnecessary loading-page transitions when cached report data is already available
  • load Plotly charts client-side in society-wide overview so SSR does not fail on window

Testing

  • cd app && bun run vitest --testTimeout 20000 src/tests/unit/pages/ReportOutput.page.test.tsx src/tests/unit/pages/report-output/ReportOutputLayout.test.tsx src/tests/unit/pages/reportBuilder/ModifyReportPage.test.tsx src/tests/unit/pages/reportBuilder/hooks/useReportBuilderState.test.tsx
  • cd app && bun run vitest --testTimeout 20000 src/tests/unit/pages/report-output/SocietyWideOverview.test.tsx src/tests/unit/pages/ReportOutput.page.test.tsx

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
policyengine-app-v2 Ready Ready Preview, Comment Apr 11, 2026 1:22am
policyengine-calculator Ready Ready Preview, Comment Apr 11, 2026 1:22am
policyengine-website Ready Ready Preview, Comment Apr 11, 2026 1:22am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
policyengine-calculator-next Skipped Skipped Apr 11, 2026 1:22am

Request Review

@anth-volk
Copy link
Copy Markdown
Collaborator Author

Closing this draft in favor of a narrower restart from main.

@anth-volk anth-volk closed this Apr 13, 2026
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.

Fix report output action flow, canonical navigation, and SSR-safe overview charts

1 participant