Merged
Conversation
Alek99
approved these changes
Oct 22, 2025
Contributor
There was a problem hiding this comment.
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_LINKconstant for the Y Combinator LinkedIn post URL - Changed
show_bannerdefault fromFalsetoTrueto display banner by default - Updated banner
hrefto use the newPOST_LINKconstant 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
1 file reviewed, no comments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.