Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
There was a problem hiding this comment.
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.
Summary
Testing