Repository navigation
Lander: keep the platform, drop the bill, for the #1225 launch - #1278
Conversation
The public page at / replaces the stub with a page for a solo developer who already pays a hosted platform. It follows the rhythm of Porter's homepage: the dashboard on a server, the root prompt handed to `ployz up`, the stack it deploys, preview environments and reviewed changes, what Ployz runs, pricing, a founder quote, a comparison and seventeen honest answers. Database moves and backups carry Soon tags. The pictures are built from the product's own Card, status colours and bottom bar, with wires in the canvas's edge shapes. The page is always light: `.light` rescopes the tokens, and the dark variant skips a `.light` subtree without changing its specificity. The glossary lets the public site say "preview environment". It launches with #1225: its CLI lines are checked against the shipped CLI before merge.
|
🚅 Environment ployz2-pr-1278 in Ployz Dashboard has no services deployed. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![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: 1f51d862a2
ℹ️ 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".
| <Line order={1} prompt="$"> | ||
| ployz up --server root@203.0.113.12 | ||
| </Line> |
There was a problem hiding this comment.
At this commit, the CLI registry in core/crates/ployz/src/cli/mod.rs:13-30 contains neither login nor up, yet the public onboarding scene presents both as a working path and shows a successful deployment. Users following the primary flow therefore receive unknown-subcommand errors; as the commit message itself says to hold this page until #1225 ships, retain or gate the stub until those commands land.
Useful? React with 👍 / 👎.
|
|
||
| export function PreviewScene() { | ||
| return ( | ||
| <div role="img" aria-label="Pull request #142 gets its own preview environment at its own https address, showing the new pricing section" inert className="relative isolate flex flex-col items-center gap-10"> |
There was a problem hiding this comment.
For screen-reader users, inert removes this element and all descendants from the accessibility tree, so the role="img" and aria-label on the same node never expose the scene description. ReviewScene repeats the pattern at line 298; place inert on an inner visual wrapper, as HeroScene does, or remove it from each labeled root.
Useful? React with 👍 / 👎.
Deploy and Sign in open sign-in in a dialog over the page again, as they did before #1063; /auth stays for direct links and before the page is interactive. /auth no longer stands alone: the _public layout gives both signed-out pages the same header, footer and light scope, and the header's links reach the lander's sections from either page. On /auth the frame drops its own sign-in links, and the sign-in panel loses its placeholder tile.
* Lander: keep the platform, drop the bill, for the #1225 launch The public page at / replaces the stub with a page for a solo developer who already pays a hosted platform. It follows the rhythm of Porter's homepage: the dashboard on a server, the root prompt handed to `ployz up`, the stack it deploys, preview environments and reviewed changes, what Ployz runs, pricing, a founder quote, a comparison and seventeen honest answers. Database moves and backups carry Soon tags. The pictures are built from the product's own Card, status colours and bottom bar, with wires in the canvas's edge shapes. The page is always light: `.light` rescopes the tokens, and the dark variant skips a `.light` subtree without changing its specificity. The glossary lets the public site say "preview environment". It launches with #1225: its CLI lines are checked against the shipped CLI before merge. * Lander: sign in over the page, and give /auth the lander's frame Deploy and Sign in open sign-in in a dialog over the page again, as they did before #1063; /auth stays for direct links and before the page is interactive. /auth no longer stands alone: the _public layout gives both signed-out pages the same header, footer and light scope, and the header's links reach the lander's sections from either page. On /auth the frame drops its own sign-in links, and the sign-in panel loses its placeholder tile.
Summary
The public lander at
/replaces the stub. It's a page for Carol, a solo developer who already pays a hosted platform, and it sells the #1225 flow. Draft until #1225 ships (see Merge Danger).One product-wide change keeps the page light when the visitor's theme is dark:
Also:
/authshares the lander's header and footer, and LoginPanel loses its placeholder "P" tile.dashboard/CONTEXT.mdlets the public site call a PR Environment a "preview environment".PloyzLogowrapper is gone.PloyzMarkstays.Evidence
/was a stub: the logo, "Deploy and run applications on servers you control." and Sign in.After: the page above, checked in Chrome at 1440px, at 390px, and under a dark system theme, where it stays light.
/authshows the shared header and footer at 1440px and 390px. From/auth, "How it works" lands on/#startclient-side.pnpm pr:check: actionlint, typecheck, lint, sdk, build and test all pass. The test stage needed one rerun: a test Postgres dropped a connection during teardown inadmission.server.postgres.test.ts, which this change doesn't touch./auth.pnpm pr:checkisn't re-run after the dialog and/authcommit (8389f37). Typecheck, lint and the data-boundaries test pass on it, and CI covers the build and tests.Merge Danger
Door: two-way
Reverting brings the stub back. Nothing here writes data.
Blast Radius: styles
dark:variant now skips.lightsubtrees, and applies again inside a.darkblock within one. The added parts sit in:where(), sodark:keeps its old specificity, (0,2,0). No product page has a.lightclass below<html>, so the product renders as before./and/authto/cloud./authfor everyone: anything that sends people to/authnow shows the lander's header and footer around the sign-in panel.ployz login,ployz up --server root@…,ployz server add root@…,ployz set,ployz deployandployz setup agent, with printed output. The CLI at this head has noup,loginorsetup. Check every terminal line and its output against the shipped CLI before merge.