[Improve] Correct Railway Composer notes after configuring the live prompts - #16
Merged
Conversation
…rompts The ROOMOTE_APP_URL deploy-screen prompt is now configured on both published templates (verified on the live deploy screens). Corrections from doing it: Railway does have a template Duplicate action, and the prompt surfaces inside the api service's Configure step rather than as a standalone top-level input.
|
No new code issues found. See task |
…y spec Both published templates already ship these; the spec and operator guide now match. SETUP_TOKEN is not optional anymore: the app refuses tokenless first-admin bootstrap outside local development, so the template must generate it. ROOMOTE_PING_BASE_URL points anonymous-analytics and version-check pings at the openmote ping service.
…ilway spec
The published templates originally repeated raw values per service, which
silently broke the one-variable custom-domain edit (web kept deriving
ROOMOTE_APP_URL from its own generated domain). Both templates now use
${{api.*}} references in the shared block per the spec, and a scratch
deploy on 2026-07-08 confirmed the two-hop references resolve to real
URLs. The design note records the verification and warns against
flattening the references again.
Contributor
Author
|
Live-template state as of this PR: both published templates (main |
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
2 participants
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.
Follow-up to #10, aligning the spec and operator guide with the published templates (main
Rj2cFo, developbP3Lsu).Composer corrections (from configuring the
ROOMOTE_APP_URLdeploy-screen prompt on both templates and verifying on the live deploy screens):Spec adoption of two variables the published templates already ship:
SETUP_TOKEN=${{secret(32)}}on api (referenced by the other app services). This is not optional anymore:apps/web/src/lib/server/setup-token.tsrefuses tokenless first-admin bootstrap outside local development, so a template without it cannot complete setup at all. The spec previously said it was intentionally unset — that note was stale against the app.ROOMOTE_PING_BASE_URL=https://ping.openmote.dev— anonymous-analytics and version-check pings (image default isping.roomote.dev); analytics opt-out stays available in the setup wizard and Settings → Misc.