Skip to content

update social marquee roles#1673

Merged
carlosabadia merged 1 commit intomainfrom
carlos/update-social-names
Oct 24, 2025
Merged

update social marquee roles#1673
carlosabadia merged 1 commit intomainfrom
carlos/update-social-names

Conversation

@carlosabadia
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Overview

Greptile Summary

Updated social testimonial cards on the landing page to display professional roles instead of platform names.

  • Changed 5 role fields from platform identifiers ("Discord", "Hacker News") to professional titles ("Data Scientist", "Software Engineer", "Product Manager", "ML Engineer", "Software Architect")
  • This provides better context about the testimonials by showing the professional background of users rather than where they posted
  • No functional changes to the code structure or logic

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The changes are purely cosmetic string updates to display data with no logic modifications, no introduction of bugs, and no violations of custom instructions
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
pcweb/pages/landing/views/social_marquee.py 5/5 Updated social testimonial role fields from platform names to professional titles

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Data as Social Data
    participant Card as social_card()
    participant Marquee as social_marquee()
    participant UI as Landing Page
    
    Dev->>Data: Update role fields in SOCIALS_1 and SOCIALS_2
    Note over Data: "Discord" → "Software Engineer"<br/>"Hacker News" → "Data Scientist"<br/>etc.
    
    UI->>Marquee: Render landing page section
    Marquee->>Card: Create cards for SOCIALS_1
    Card->>Data: Read Social dataclass instances
    Data-->>Card: Return updated role values
    Card-->>Marquee: Return social card components
    
    Marquee->>Card: Create cards for SOCIALS_2
    Card->>Data: Read Social dataclass instances
    Data-->>Card: Return updated role values
    Card-->>Marquee: Return social card components
    
    Marquee-->>UI: Display marquee with updated roles
Loading

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@carlosabadia carlosabadia merged commit ac2e460 into main Oct 24, 2025
12 of 14 checks passed
@carlosabadia carlosabadia deleted the carlos/update-social-names branch October 24, 2025 00:48
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.

2 participants