Skip to content

Conversation

@carlosabadia
Copy link
Collaborator

No description provided.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 29, 2026

Greptile Overview

Greptile Summary

This PR updates the sign-in link in the navbar component to use the new authentication flow. The change switches from using REFLEX_CLOUD_URL to REFLEX_BUILD_URL as the base URL, updates the endpoint from the root path to /cloud-login, changes the query parameter from redirect_url to redirect_to, and specifies /callback/ as the redirect destination. The unused REFLEX_CLOUD_URL import was also removed from the file.

Confidence Score: 5/5

  • This PR is safe to merge with no concerns
  • The change is straightforward and well-contained. It updates a single sign-in URL to align with a new authentication flow. The unused import is properly cleaned up. No logical errors, security issues, or breaking changes are introduced.
  • No files require special attention

Important Files Changed

Filename Overview
pcweb/components/docpage/navbar/navbar.py Updated sign-in link from REFLEX_CLOUD_URL to REFLEX_BUILD_URL with new cloud-login path and redirect_to parameter

Sequence Diagram

sequenceDiagram
    participant User
    participant Navbar
    participant BuildServer as build.reflex.dev
    
    User->>Navbar: Clicks "Sign In" button
    Navbar->>BuildServer: Redirect to /cloud-login?redirect_to=.../callback/
    BuildServer->>BuildServer: Authenticate user
    BuildServer->>BuildServer: Redirect to /callback/
    BuildServer->>User: Complete authentication flow
Loading

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.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@Kastier1 Kastier1 merged commit 483db4a into main Jan 29, 2026
10 checks passed
@Kastier1 Kastier1 deleted the carlos/update-sign-in-link branch January 29, 2026 23:30
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