Skip to content

Remove PE84 promo banner from homepage#908

Merged
MaxGhenis merged 1 commit intomainfrom
codex/remove-pe84-banner
Apr 3, 2026
Merged

Remove PE84 promo banner from homepage#908
MaxGhenis merged 1 commit intomainfrom
codex/remove-pe84-banner

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • remove the temporary PE84 banner from the country homepage
  • restore the normal homepage flow so the shared header sits directly above the hero

Testing

  • bun run --filter=@policyengine/website build:next

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 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 3, 2026 9:40am
policyengine-calculator Ready Ready Preview, Comment Apr 3, 2026 9:40am
policyengine-website Ready Ready Preview, Comment Apr 3, 2026 9:40am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
policyengine-calculator-next Skipped Skipped Apr 3, 2026 9:40am

Request Review

Copy link
Copy Markdown

@policyengine policyengine bot left a comment

Choose a reason for hiding this comment

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

The PR successfully removes the PE84 banner from the homepage. The change is minimal and correct:

✅ Removed the import and usage of <PE84Banner /> from the homepage
✅ HeroSection now appears first, restoring the normal flow
✅ No other references to PE84Banner in the codebase (besides the banner file itself)

Minor suggestion (non-blocking): The website/src/components/home/PE84Banner.tsx file still exists but is now unused. Consider deleting it in a follow-up to clean up dead code. The PE84 page itself (/pe84) is intentionally kept, which is correct.

The build command mentioned in the PR description should work fine since this is a straightforward removal with no logic changes.

@MaxGhenis MaxGhenis merged commit 2f4e8da into main Apr 3, 2026
13 checks passed
@MaxGhenis MaxGhenis deleted the codex/remove-pe84-banner branch April 3, 2026 11:26
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.

1 participant