Skip to content

update banner#1670

Merged
carlosabadia merged 1 commit intomainfrom
carlos/launch-banner
Oct 22, 2025
Merged

update banner#1670
carlosabadia merged 1 commit intomainfrom
carlos/launch-banner

Conversation

@carlosabadia
Copy link
Collaborator

No description provided.

@carlosabadia carlosabadia merged commit d6bcb3c into main Oct 22, 2025
9 of 10 checks passed
@carlosabadia carlosabadia deleted the carlos/launch-banner branch October 22, 2025 17:33
Copy link
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

Summary

Updated the hosting banner to promote Reflex Build via a Y Combinator LinkedIn post, replacing the previous direct link to build.reflex.dev. The banner is now shown by default to users.

Changes:

  • Added POST_LINK constant for the Y Combinator LinkedIn post URL
  • Changed show_banner default from False to True to display banner by default
  • Updated banner href to use the new POST_LINK constant instead of the hardcoded build.reflex.dev URL

Confidence Score: 5/5

  • This PR is safe to merge with no risks
  • The changes are minimal and straightforward - only updating a URL constant, changing a boolean default value, and using the constant. No logic changes, no new dependencies, and follows existing patterns in the codebase. All custom instructions are satisfied.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
pcweb/components/hosting_banner.py 5/5 Updated banner link to Y Combinator LinkedIn post and enabled banner visibility by default

Sequence Diagram

sequenceDiagram
    participant User
    participant HostingBanner
    participant HostingBannerState
    participant Browser

    User->>HostingBanner: Page Load
    HostingBanner->>HostingBannerState: Check show_banner (default: True)
    alt show_banner is True
        HostingBannerState->>HostingBanner: Return True
        HostingBanner->>Browser: Render banner with YC LinkedIn link
        User->>HostingBanner: Click banner link
        HostingBanner->>Browser: Navigate to POST_LINK (external)
        User->>HostingBanner: Click X icon
        HostingBanner->>HostingBannerState: Call hide_banner()
        HostingBannerState->>HostingBannerState: Set show_banner = False
        HostingBannerState->>HostingBanner: Update state
        HostingBanner->>Browser: Hide banner
    else show_banner is False
        HostingBannerState->>HostingBanner: Return False
        HostingBanner->>Browser: Do not render banner
    end
Loading

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

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

Comments