Skip to content

Update demo URLs to demo.hotcrm.com#336

Merged
hotlong merged 1 commit intomainfrom
claude/update-demo-url-to-demo-hotcrm-com
Apr 14, 2026
Merged

Update demo URLs to demo.hotcrm.com#336
hotlong merged 1 commit intomainfrom
claude/update-demo-url-to-demo-hotcrm-com

Conversation

@Claude
Copy link
Copy Markdown
Contributor

@Claude Claude AI commented Apr 14, 2026

Description

Updated homepage CTA buttons to point to the live demo instance at https://demo.hotcrm.com instead of non-existent local routes.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • CI/CD update

Related Issues

N/A

Changes Made

  • Updated Hero Section CTA buttons (/signup and /book-demohttps://demo.hotcrm.com)
  • Updated Final CTA section buttons to point to live demo URL
  • Total of 4 button links updated across the homepage

Testing

  • Unit tests pass (npm test)
  • Linting passes (npm run lint)
  • Build succeeds (npm run build)
  • Manual testing completed
  • New tests added (if applicable)

Screenshots

N/A

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Additional Notes

All homepage call-to-action buttons now direct users to the centralized demo environment. This provides a consistent user experience for trial and demo access.

Agent-Logs-Url: https://github.com/objectstack-ai/hotcrm/sessions/3d26a726-586f-47a5-a01d-5a988e4db9fe

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hotcrm-demo Ready Ready Preview, Comment Apr 14, 2026 9:06am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
hotcrm Ignored Ignored Apr 14, 2026 9:06am

Request Review

@hotlong hotlong marked this pull request as ready for review April 14, 2026 09:06
Copilot AI review requested due to automatic review settings April 14, 2026 09:06
@hotlong hotlong merged commit 6985b88 into main Apr 14, 2026
6 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the HotCRM docs homepage CTAs to route users to the live demo environment (https://demo.hotcrm.com) rather than local, non-existent routes, improving the trial/demo flow for the marketing site.

Changes:

  • Updated Hero section CTA links to https://demo.hotcrm.com.
  • Updated Final CTA section links to https://demo.hotcrm.com.

Comment on lines 62 to 71
<Link
href="/signup"
href="https://demo.hotcrm.com"
className="hero-cta-primary inline-flex h-12 items-center justify-center rounded-lg bg-primary px-8 text-base font-medium text-primary-foreground shadow-lg transition-all hover:shadow-primary/25 hover:shadow-xl"
>
{dict.home.getStarted}
<ArrowRight className="ml-2 w-5 h-5" />
</Link>
<Link
href="/book-demo"
href="https://demo.hotcrm.com"
className="inline-flex h-12 items-center justify-center rounded-lg border border-border bg-background/50 backdrop-blur-sm px-8 text-base font-medium shadow-sm transition-all hover:bg-accent hover:text-accent-foreground"
Copy link

Copilot AI Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://demo.hotcrm.com is duplicated across multiple CTA links in this file. Consider extracting it into a single constant (or config/env value) and referencing that constant in each href to reduce the chance of future inconsistencies when the demo URL changes again.

Copilot uses AI. Check for mistakes.
Comment on lines 369 to 376
{dict.home.readDocs}
<ArrowRight className="ml-2 w-5 h-5" />
</Link>
<Link
href="/book-demo"
href="https://demo.hotcrm.com"
className="inline-flex h-12 items-center justify-center rounded-lg border border-border bg-background/50 backdrop-blur-sm px-8 text-base font-medium shadow-sm transition-all hover:bg-accent hover:text-accent-foreground"
>
{dict.home.starOnGithub}
Copy link

Copilot AI Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The i18n keys used for these CTAs are misleading (viewOnGithub, readDocs, starOnGithub) given the button text (e.g. “Book a Demo”, “Start Free Trial”, “Talk to Sales”) and now point to the demo URL. Consider renaming these dictionary keys to reflect their intent (and updating references) to reduce the chance of wiring the wrong destination in future edits.

Copilot generated this review using guidance from organization custom instructions.
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.

3 participants