Skip to content

Read routes from _unevaluated_pages#5503

Merged
adhami3310 merged 1 commit into
mainfrom
masenf/fix-on-load-uneval
Jun 26, 2025
Merged

Read routes from _unevaluated_pages#5503
adhami3310 merged 1 commit into
mainfrom
masenf/fix-on-load-uneval

Conversation

@masenf
Copy link
Copy Markdown
Collaborator

@masenf masenf commented Jun 26, 2025

On the backend, _pages may be empty if the compile was skipped.

On the backend, _pages may be empty if the compile was skipped.
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.

PR Summary

Modified backend routing logic in reflex/app.py to use _unevaluated_pages instead of _pages for route generation, ensuring routes are properly handled even when compilation is skipped.

  • Updated App.router property to use _unevaluated_pages for reliable route mapping when _pages is empty due to skipped compilation

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jun 26, 2025

CodSpeed Performance Report

Merging #5503 will not alter performance

Comparing masenf/fix-on-load-uneval (526d5a2) with main (b816328)

Summary

✅ 8 untouched benchmarks

@adhami3310 adhami3310 merged commit 00091a9 into main Jun 26, 2025
40 of 41 checks passed
@adhami3310 adhami3310 deleted the masenf/fix-on-load-uneval branch June 26, 2025 03:19
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