Skip to content

Add Try tab with a page for trial launch on Virtuozzo Cloud#9

Merged
il10241024 merged 1 commit into
constructorfabric:mainfrom
ihorman:main
Jun 22, 2026
Merged

Add Try tab with a page for trial launch on Virtuozzo Cloud#9
il10241024 merged 1 commit into
constructorfabric:mainfrom
ihorman:main

Conversation

@ihorman

@ihorman ihorman commented Jun 22, 2026

Copy link
Copy Markdown

What changed

Pages to review

  • index.html
  • foundation.html
  • elements.html
  • learn.html
  • participate.html
  • privacy-policy.html
  • 404.html
  • styles.css
  • partials.jsx
  • assets/
  • Other:

Preview

A preview URL will be posted automatically by GitHub Actions after CI succeeds. The format is:

https://constructorfabric.github.io/website-previews/pr-<NUMBER>/

If you do not see a preview URL on this PR within a few minutes of pushing, see docs/pr-preview-process.md for troubleshooting.

Screenshots

Required for visual changes. Optional for text-only changes.

Desktop:

Mobile:

Checklist

  • I checked the site locally (Live Preview extension or python3 -m http.server)
  • I listed every affected page above
  • I added screenshots for visual changes
  • I avoided unrelated formatting changes
  • I followed the banned-words and no-emojis conventions in CONTRIBUTING.md

Summary by CodeRabbit

  • New Features
    • Added a new "Try" page accessible from the site's primary navigation and footer.
    • Introduced an interactive trial form allowing users to enter their work email for registration.
    • Added a preview card displaying trial offerings and key benefits.

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Adds a new try.html page that renders a "Try Constructor Fabric" trial landing flow using inline React/Babel. The page includes email-based registration form logic, Virtuozzo URL construction helpers, static preview and layout components, and two new navigation links (header and footer) in partials.jsx.

Changes

Try Page Addition

Layer / File(s) Summary
Header and footer navigation links
partials.jsx
Adds a "Try" entry to SiteHeader's nav items array and a "Try" link to SiteFooter's Resources list, both pointing to try.html.
Page shell, styles, and React environment
try.html
Defines the HTML document with Open Graph metadata, CDN script tags for React/ReactDOM/Babel, inline CSS for the trial panel/form/grid layout, the #root mount point, and Babel environment constants (app id, JPS manifest URL, platform base URL, group id).
URL construction helpers and TrialForm
try.html
Implements registrationUrl and buildTrialUrl that compose the Virtuozzo deployment URL. Defines TrialForm with email validation, fetch GET submission, JSON response parsing against nested result fields, success/error status display, and loading/disabled button state.
TrialPreview, TryPage composition, and mount
try.html
Adds TrialPreview static card, composes TryPage with SiteHeader, hero section, TrialPreview, TrialForm, confirmation section, CtaStart, and SiteFooter, then mounts into #root.

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant TrialForm
  participant buildTrialUrl
  participant VirtauzzoAPI

  User->>TrialForm: enter email and submit
  TrialForm->>TrialForm: validate email format
  alt invalid email
    TrialForm-->>User: show validation error status
  else valid email
    TrialForm->>buildTrialUrl: pass email
    buildTrialUrl-->>TrialForm: URL with email, appid, jps, group, referrer params
    TrialForm->>VirtauzzoAPI: GET constructed trial URL
    VirtauzzoAPI-->>TrialForm: response body text
    TrialForm->>TrialForm: attempt JSON parse, check result.result / result.error
    alt success
      TrialForm-->>User: display success message
    else failure
      TrialForm-->>User: display composed error message
    end
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 Hop, hop, a new page appears,
A "Try" link in the nav now steers,
An email typed, a URL spun,
Virtuozzo registers — it's done!
The rabbit stamps the deploy run. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is a blank template with no actual content filled in. Required sections like 'What changed', 'Pages to review', and checklist items are all uncompleted. Complete the 'What changed' section describing the new Try page, mark 'partials.jsx' and 'try.html' as affected in 'Pages to review', add screenshots if applicable, and check the relevant checklist items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding a new 'Try' tab and page for trial launch on Virtuozzo Cloud, which matches the additions to partials.jsx and the new try.html file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@constructorfabric

Copy link
Copy Markdown

Preview deployment is ready:

https://constructorfabric.github.io/website-previews/pr-9/

This preview is built from the latest commit on this PR. It is regenerated automatically on every push. Old previews are cleaned up after 30 days.

@il10241024 il10241024 merged commit 2726264 into constructorfabric:main Jun 22, 2026
1 of 3 checks passed
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.

2 participants