Skip to content

draft: preview mode follow ups#9195

Draft
royendo wants to merge 2 commits intomainfrom
royendo/preview-mode-route-utils
Draft

draft: preview mode follow ups#9195
royendo wants to merge 2 commits intomainfrom
royendo/preview-mode-route-utils

Conversation

@royendo
Copy link
Copy Markdown
Contributor

@royendo royendo commented Apr 7, 2026

  1. AppMode enum (replacing string literals)
  2. Centralized route utility functions with tests (all 8 tests passing, full test suite green)

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

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.

1 participant