[chores] Deploy: added get early access link#176
Conversation
WalkthroughThe deploy page is updated to replace internal contact button components with direct external links to a Jotform sign-up form. The change removes the unused contact_button template import and updates all AWS, Azure, Google Cloud, and DigitalOcean deployment option modals to display "Get Early Access" buttons linking directly to the form. Modal descriptive text is also updated to reference "Get notified" instead of contacting the team. Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@content/pages/deploy.html`:
- Around line 52-53: Several external JotForm anchor tags (e.g., the "Get
notified" links) use target="_blank" without rel attributes, enabling reverse
tabnabbing; update every anchor that opens an external JotForm in a new tab (all
occurrences in the AWS, Azure, Google Cloud, and DigitalOcean modals and the
duplicate link near the top) to include rel="noopener noreferrer" alongside
target="_blank" so the opened pages cannot access window.opener.
🧹 Nitpick comments (1)
content/pages/deploy.html (1)
37-69: All four modal blocks are nearly identical — consider extracting a reusable template/partial.The modal markup (body text + footer CTA) is copy-pasted across AWS, Azure, Google Cloud, and DigitalOcean with only the header title differing. The Jotform URL alone is hardcoded 8 times. If the form URL or CTA text changes, all instances must be updated in lockstep, which is error-prone.
If the templating engine supports it, extracting a shared partial/include parameterized by the provider name would reduce duplication.
Also applies to: 84-114, 129-159, 174-204
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/pages/deploy.html
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
| <a href="https://form.jotform.com/260416646535055" target="_blank">Get notified</a> | ||
| the moment they are ready for deployment. |
There was a problem hiding this comment.
Add rel="noopener noreferrer" to all target="_blank" links.
All eight external Jotform links open in a new tab via target="_blank" but lack rel="noopener noreferrer". This exposes the page to reverse tabnabbing where the opened page can access window.opener. The same issue applies to the identical links in the Azure (lines 97-98, 103-106), Google Cloud (lines 142-143, 148-151), and DigitalOcean (lines 187-188, 193-196) modals.
Proposed fix (showing AWS modal; apply same pattern to all four)
- <a href="https://form.jotform.com/260416646535055" target="_blank">Get notified</a>
+ <a href="https://form.jotform.com/260416646535055" target="_blank" rel="noopener noreferrer">Get notified</a>
the moment they are ready for deployment.- <a class="button with-text is-orange" href="https://form.jotform.com/260416646535055" target="_blank">
+ <a class="button with-text is-orange" href="https://form.jotform.com/260416646535055" target="_blank" rel="noopener noreferrer">Also applies to: 58-61
🤖 Prompt for AI Agents
In `@content/pages/deploy.html` around lines 52 - 53, Several external JotForm
anchor tags (e.g., the "Get notified" links) use target="_blank" without rel
attributes, enabling reverse tabnabbing; update every anchor that opens an
external JotForm in a new tab (all occurrences in the AWS, Azure, Google Cloud,
and DigitalOcean modals and the duplicate link near the top) to include
rel="noopener noreferrer" alongside target="_blank" so the opened pages cannot
access window.opener.
Added link to "get early access" form.