OPS-832: Marketing Goverment Page

[carlosabadia]

No description provided.

[linear]

OPS-832 Convert Government page

[greptile-apps]

Greptile Overview

Greptile Summary

This PR adds a new marketing page for government use cases at /use-cases/government. The implementation follows established patterns from the finance and healthcare pages, reusing common components for hero, features, stats, FAQ, and final CTA sections.

Key Changes:

• Created new government use case page with 10 view components
• Added 6 new SVG logo assets for government/public sector organizations
• Updated navbar to include government option in Solutions menu
• Registered new page route in pages __init__.py

Issues Found:

• The quotes.py file was created but is never imported or used in the main page - this appears to be dead code that should either be integrated or removed
• The unused quotes component contains testimonials from financial/private sector companies (World Bank, Bayesline, Crédit Agricole) rather than government agencies, suggesting it may have been copied from another page without proper customization

Confidence Score: 3/5

• Safe to merge with minor cleanup - unused code should be addressed
• The implementation is solid and follows established patterns throughout the codebase. However, the unused quotes.py file represents dead code that violates best practices. The page will function correctly without it, but leaving unused files can cause confusion and maintenance issues.
• Pay attention to pcweb/pages/use_cases/government/views/quotes.py - decide whether to integrate it into the page or remove it entirely

Important Files Changed

File Analysis

Filename	Score	Overview
pcweb/pages/use_cases/government/init.py	3/5	New government use case page created with standard structure, but missing quotes section that exists in similar pages
pcweb/pages/use_cases/government/views/quotes.py	2/5	Quotes component created but not imported or used in main page - appears to be unused code
pcweb/pages/init.py	5/5	Added government use case page import, consistent with existing patterns
pcweb/components/docpage/navbar/navbar.py	5/5	Added government page to navbar Solutions menu

Sequence Diagram

sequenceDiagram
    participant User
    participant Browser
    participant Server
    participant GovernmentPage
    participant ViewComponents
    participant CommonComponents
    
    User->>Browser: Navigate to /use-cases/government
    Browser->>Server: Request page
    Server->>GovernmentPage: Render government_use_case_page()
    
    GovernmentPage->>ViewComponents: Call hero()
    ViewComponents->>CommonComponents: Use left_content(), right_content()
    CommonComponents-->>ViewComponents: Return hero content
    ViewComponents-->>GovernmentPage: Return hero section
    
    GovernmentPage->>ViewComponents: Call social_proof()
    ViewComponents->>CommonComponents: Use logos_carousel(LOGOS)
    CommonComponents-->>ViewComponents: Return social proof content
    ViewComponents-->>GovernmentPage: Return social proof section
    
    GovernmentPage->>ViewComponents: Call text_section_1()
    ViewComponents->>CommonComponents: Use text_section()
    CommonComponents-->>ViewComponents: Return text content
    ViewComponents-->>GovernmentPage: Return text section
    
    GovernmentPage->>ViewComponents: Call features_1()
    ViewComponents->>CommonComponents: Use feature_card()
    CommonComponents-->>ViewComponents: Return feature cards
    ViewComponents-->>GovernmentPage: Return features section
    
    GovernmentPage->>ViewComponents: Call text_section_2()
    GovernmentPage->>ViewComponents: Call features_2()
    GovernmentPage->>ViewComponents: Call text_section_3()
    GovernmentPage->>ViewComponents: Call stats()
    GovernmentPage->>ViewComponents: Call faq()
    GovernmentPage->>ViewComponents: Call final_section()
    
    Note over GovernmentPage,ViewComponents: quotes() exists but is NOT called
    
    GovernmentPage-->>Server: Complete page render
    Server-->>Browser: Return HTML/JS
    Browser-->>User: Display government page

Loading

[greptile-apps]

_{2 files reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Additional Comments (1)

pcweb/pages/use_cases/government/views/quotes.py
this entire file is unused - quotes() is not imported or called in __init__.py. The finance page includes quotes between text sections, but government page omits it. Either import and use this component, or remove the file entirely.

Prompt To Fix With AI

This is a comment left during a code review.
Path: pcweb/pages/use_cases/government/views/quotes.py
Line: 1:34

Comment:
this entire file is unused - `quotes()` is not imported or called in `__init__.py`. The finance page includes quotes between text sections, but government page omits it. Either import and use this component, or remove the file entirely.

How can I resolve this? If you propose a fix, please make it concise.

[tgberkeley]

have this section be in bullet points