ENG-8503: finance landing

[carlosabadia]

No description provided.

[linear]

ENG-8503 Implement reflex web landing

[greptile-apps]

Greptile Overview

Greptile Summary

This PR creates a new finance-focused landing page for /use-cases/finance, replacing the previous implementation with a modular, component-based approach. The changes include:

• New modular structure: Refactored finance landing page into separate view components (hero, features, quotes, stats, social proof, text sections, final CTA)
• Visual assets: Added company logos (Bayesline, Credit Agricole, Man Group, World Bank) in both light and dark theme variants
• Design system updates: Introduced marketing color palette (m-slate and m-violet) for consistent theming across the page
• Code organization: Moved from monolithic finance_landing.py to organized finance/__init__.py module with dedicated view files

Critical issue found: The quotes section (quotes.py) contains duplicate placeholder content across all 4 testimonial cards - they all show identical text including "FirstName LastName, Crédit Agricole CIB". This needs to be updated with actual unique testimonials before launch.

Confidence Score: 3/5

• This PR has solid architecture and no functional bugs, but contains placeholder content that must be replaced before production
• Score of 3 reflects good code quality and structure, but the duplicate placeholder testimonials in quotes.py is a critical content issue that would be embarrassing if deployed to production. The technical implementation is clean and follows best practices, but the incomplete content prevents this from being merge-ready.
• Pay close attention to pcweb/pages/use_cases/finance/views/quotes.py - all testimonial content needs to be replaced with real data

Important Files Changed

File Analysis

Filename	Score	Overview
pcweb/pages/use_cases/finance/views/quotes.py	1/5	Duplicate placeholder content for all 4 testimonial cards - needs unique content for each
pcweb/pages/use_cases/finance/init.py	5/5	Clean page composition importing all view components and assembling the finance landing page
pcweb/pages/use_cases/finance/views/hero.py	5/5	Hero section with CTA buttons and feature list - properly structured with responsive design
pcweb/pages/use_cases/finance/views/features_1.py	5/5	Feature cards describing AI tools, legacy modernization, workflow automation - clean implementation
pcweb/pages/use_cases/finance/views/features_2.py	5/5	Additional feature cards with stats highlighting speed, Python-first approach, enterprise deployment
pcweb/pages/use_cases/finance/views/final_section.py	5/5	Call-to-action section with demo scheduling and builder link - uses constants properly

Sequence Diagram

sequenceDiagram
    participant User
    participant Router
    participant MainPage as mainpage decorator
    participant FinancePage as finance_use_case_page
    participant Hero as hero()
    participant SocialProof as social_proof()
    participant Features1 as features_1()
    participant Quotes as quotes()
    participant Features2 as features_2()
    participant Stats as stats()
    participant FinalSection as final_section()
    
    User->>Router: Navigate to /use-cases/finance
    Router->>MainPage: Route matches path
    MainPage->>FinancePage: Call finance_use_case_page()
    FinancePage->>Hero: Render hero section
    Hero-->>FinancePage: Hero component with CTA buttons
    FinancePage->>SocialProof: Render social proof
    SocialProof-->>FinancePage: Company logos grid
    FinancePage->>Features1: Render first feature set
    Features1-->>FinancePage: 4 feature cards
    FinancePage->>Quotes: Render testimonials
    Note over Quotes: ISSUE: All 4 cards have<br/>duplicate placeholder content
    Quotes-->>FinancePage: Quote cards (duplicated)
    FinancePage->>Features2: Render second feature set
    Features2-->>FinancePage: 4 feature cards with stats
    FinancePage->>Stats: Render customer stats
    Stats-->>FinancePage: Stat cards grid
    FinancePage->>FinalSection: Render CTA section
    FinalSection-->>FinancePage: Demo booking + builder link
    FinancePage-->>MainPage: Complete page component
    MainPage-->>Router: Rendered page
    Router-->>User: Display finance landing page

Loading

[greptile-apps]

_{21 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[tgberkeley]

Let's turn the logos for finance to also be a carosel. Let's add the logos twosigma.com, ramp, capitalone.com and ubs.com