Skip to content

Restore OBBBA household explorer to use Streamlit app#2698

Merged
MaxGhenis merged 3 commits into
masterfrom
fix-obbba-streamlit-embed
Jul 24, 2025
Merged

Restore OBBBA household explorer to use Streamlit app#2698
MaxGhenis merged 3 commits into
masterfrom
fix-obbba-streamlit-embed

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • Updates apps.json to point obbba-household-explorer to the Streamlit URL
  • Removes the redirect component and route that was incorrectly added

Context

The OBBBA household explorer should embed the Streamlit app, not redirect to a different slug. This PR restores the original functionality.

Test plan

  • Visit /us/obbba-household-explorer and verify it displays the Streamlit app in an iframe
  • Confirm there's no redirect happening
  • Check that the Streamlit app loads and functions correctly

🤖 Generated with Claude Code

MaxGhenis and others added 2 commits July 23, 2025 22:37
Rename blog post and image files back to introducing-obbba-household-explorer to fix broken homepage link. This maintains the existing blog post about the Streamlit app.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Point obbba-household-explorer slug to the Streamlit URL
- Remove redirect component and route
- This ensures the app embeds the Streamlit app as originally intended

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 24, 2025 02:44
@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
policyengine-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 24, 2025 2:51am

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR restores the OBBBA household explorer to use the Streamlit app instead of redirecting to a different URL. The changes reverse a previous redirect implementation and update references to use the correct slug and Streamlit URL.

  • Updates the apps.json configuration to point to the Streamlit app URL with the correct slug
  • Removes the redirect component and routing that was incorrectly redirecting users
  • Updates post references to use the correct filename and image names

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

File Description
src/routing/RedirectOBBBAHouseholdExplorer.jsx Removes the entire redirect component that was redirecting to a different slug
src/posts/posts.json Updates filename and image references from "household-by-household" to "household-explorer"
src/apps/apps.json Updates URL to Streamlit app and changes slug from "obbba-household-by-household" to "obbba-household-explorer"
src/PolicyEngine.jsx Removes import and route for the redirect component

- Keep obbba-household-explorer pointing to Streamlit app with embed parameter
- Add new obbba-household-by-household entry pointing to GitHub Pages app
- This provides access to both versions of the OBBBA analysis tool

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@MaxGhenis MaxGhenis merged commit 5bc1d77 into master Jul 24, 2025
4 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in policyengine-app Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants