Restore VForge original while preserving Forge Live - #103
Conversation
Return the product, marketing site, workspace, auth and full module surface to the ba27b48 baseline. Preserve the scoped three-view live portal, invitations, comments, SSE, owned Hetzner API and MetaMCP security boundary. Remove only the later product-shell replacement.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1d9e37d224
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <Labs /> | ||
| <CTA /> | ||
| </main> | ||
| <MarketingFooter /> |
There was a problem hiding this comment.
On the restored production landing, submitting either the Partners or Asociados form only calls preventDefault() and setSent(true) in MarketingFooter.tsx; no request or persistence occurs, yet the visitor sees “Mensaje recibido,” so every submitted lead is silently discarded. Connect the form to a real endpoint/database and show success only after it confirms receipt.
AGENTS.md reference: AGENTS.md:L134-L136
Useful? React with 👍 / 👎.
| return <ProductHome />; | ||
| return ( | ||
| <div className="min-h-screen" style={{ background: "#050a14" }}> | ||
| <MarketingHeader /> |
There was a problem hiding this comment.
At mobile widths such as 390px, this restores MarketingHeader, whose single non-wrapping row simultaneously displays the logo, four navigation links, GitHub, login, and signup controls without any media query or collapse behavior. Their combined width exceeds the viewport, and the global overflow-x: hidden clips the right-hand authentication controls, making the primary signup path inaccessible on phones.
Useful? React with 👍 / 👎.
| return <ProductHome />; | ||
| return ( | ||
| <div className="min-h-screen" style={{ background: "#050a14" }}> | ||
| <MarketingHeader /> |
There was a problem hiding this comment.
Rendering the legacy marketing components exposes prominent /pricing links in the header, footer, and CTA, but a repo-wide route search finds no app/pricing/page.tsx or other handler for that path. Any visitor selecting “Precios” or “Ver precios” therefore lands on a 404; restore the route or point these links to an existing billing destination.
Useful? React with 👍 / 👎.
|
|
||
| return <ProductHome />; | ||
| return ( | ||
| <div className="min-h-screen" style={{ background: "#050a14" }}> |
There was a problem hiding this comment.
The restored landing root hardcodes #050a14, so changes to the design-system background or theme cannot propagate to this page. Use the appropriate CSS variable instead of introducing a separate production color source.
AGENTS.md reference: AGENTS.md:L82-L84
Useful? React with 👍 / 👎.
1 participant
Corrección precisa
Este PR revierte únicamente el reemplazo de interfaz introducido después de
ba27b488.ProductHomeyProductShell, que habían convertido el visor en sustituto del productoAlcance verificado
Solo cambian 6 archivos de UI:
app/app/layout.tsxapp/app/page.tsxapp/layout.tsxapp/page.tsxcomponents/marketing/ProductHome.tsxycomponents/workspace/ProductShell.tsxNo cambia la base de datos, la API, MetaMCP, Hetzner, secrets, migraciones ni rutas de Forge Live.
Baseline
Árbol exacto de
ba27b488:5c3a97cfb82f6cc5487ee2f5e2e8c5250b062b33.