Refactor product landing pages: content, pricing, and dashboard UX#115
Refactor product landing pages: content, pricing, and dashboard UX#115
Conversation
…ests and improve feature descriptions. Adjust sidebar navigation in GraphDashboard for better organization and add semantic search capabilities in SchemaArchitecture.
|
@claude please review this PR |
|
Claude finished @jfrench9's task in 2m 6s —— View job PR Review: Refactor product landing pages
Overall this is a clean, well-scoped PR. The
|
| Tier | Old Price | New Price | Old Credits | New Credits |
|---|---|---|---|---|
| Standard | $0 | $99 | 1,000 | 8,000 |
| Large | $49 | $299 | 10,000 | 32,000 |
| XLarge | $199 | $699 | 50,000 | 100,000 |
Two concerns:
-
Missing test assertion for XLarge credits: The updated test only checks
8,000 AI credits/month(Standard) and32,000 AI credits/month(Large). The100,000 AI credits/monthassertion for XLarge was dropped without a clear reason. -
Dedicated support→Priority support: XLarge previously had "Dedicated support" as a differentiator. Now all paid tiers have "Priority support." The old test assertion for this was simply removed rather than updated. IfDedicated supportwas a meaningful product differentiator, this should be an intentional decision, not a silent removal. -
Test mocks vs. real API data: If these tier mocks are meant to reflect real backend data, the pricing changes here (especially removing the free Standard tier) are substantial. Confirm these align with what
GET /tiersactually returns.
FinancialServices.tsx — Bug fix (pre-existing, resolved by this PR)
The old code had a CSS class string bug that caused a broken class:
// Old code (buggy - missing space before template literal):
className={`hidden shrink-0 rounded-md px-2.5 py-1 text-xs font-medium xs:block${
item.status === 'Active' ? ...`}
// Rendered as: "...xs:blockbg-green-500/20 text-green-400"The new code removes the status badge entirely, so this bug is gone. Worth noting this was a silent fix.
FeaturesGrid.tsx — Feature category renamed (Orchestration → Open Source)
The "Orchestration" section (previously describing Dagster pipelines) has been replaced with "Open Source" (Docker/CloudFormation/GitHub Actions). This removes any mention of Dagster from the landing page. Confirm this is an intentional product positioning decision — if Dagster is still part of the platform, it may need a home elsewhere.
Minor Issues
BusinessServices.tsx— The heading paragraph still ends without a period:"queryable knowledge graph with semantic search and AI memory". The original also lacked a period, so not a regression.GraphDashboard.tsx—key={idx}insidebarItems.map(): Using array index as key is acceptable here since the list is static and never reorders.FinancialServices.tsx—h-fullon card wrappers: Good fix; ensures equal-height cards in a grid layout.SchemaArchitecture.tsx— "Semantic Hybrid Search" and BM25 + KNN descriptions are more precise than before. Solid improvement.
Summary
The refactor is clean and the net line reduction comes from legitimate simplification (inline SVGs → icon components). The main items to confirm before merging:
- Is removing
Billingfrom the sidebar intentional for this demo component? - Do the new tier prices in test mocks reflect actual backend tier data?
- Is the Orchestration/Dagster feature intentionally removed from the features grid?
|
Summary
This PR aligns and enhances several product-facing landing page components, improving content clarity, pricing details, and dashboard organization. The changes span 8 files with a net reduction of ~65 lines, reflecting a consolidation and cleanup effort across platform, pricing, and dashboard components.
Changes
Platform & Landing Content
platform/content.tsx: Updated platform content copy for better alignment with current product positioningpricing/content.tsx: Enhanced pricing tier details with more descriptive feature breakdowns and adjusted tier informationFeaturesGrid.tsx: Improved feature descriptions in the landing page grid for clearer value communicationPlatform Components
BusinessServices.tsx: Refined business services feature descriptions and content hierarchyFinancialServices.tsx: Updated financial services section with improved copy and structural adjustmentsGraphDashboard.tsx: Significant refactor (~418 lines touched) — reorganized sidebar navigation for better information architecture and improved overall component structure, resulting in a net reduction of ~65 linesSchemaArchitecture.tsx: Added semantic search capabilities to the schema architecture componentTests
TierSelectionStep.test.tsx: Updated tier selection step tests to reflect new pricing content and tier descriptions, ensuring test assertions match the revised copyKey UI/UX Improvements
GraphDashboardhas been reorganized for more intuitive grouping and discoverabilitySchemaArchitecturenow surfaces semantic search capabilities, improving schema exploration UXBreaking Changes
None anticipated. All changes are content/UI refinements and internal component restructuring. No API contracts, prop interfaces, or routing changes detected.
Testing Notes for Reviewers
/platform,/pricing) for layout integrity and content accuracyTierSelectionStep.test.tsx— tests have been updated but verify they pass:npm test -- --testPathPattern=TierSelectionStepGraphDashboardgiven the significant restructuringBrowser Compatibility
No new browser APIs or CSS features introduced. Changes are primarily content and component structure updates. Standard compatibility expected across:
🤖 Generated with Claude Code
Branch Info:
refactor/product-landing-alignmentmainCo-Authored-By: Claude noreply@anthropic.com