The app page title now shows the app's configured branch as a small neutral monospace label next to the status badge. You can read it there without opening the edit page. The line under the title now shows only the repository; it used to read repo@branch.
The app list is unchanged. It already has a Branch column, and the owner withdrew that part of the issue.
A new test, internal/handlers/app_branch_test.go, renders the app page for an app on branch staging. It checks that the branch is in the title row with the status badge.
Every class used is already in the compiled static/css/tailwind.css, so the stylesheet is unchanged.
Judgement call: the branch label gets break-all, so a very long branch name wraps at phone width instead of overflowing. It also gets a Branch tooltip.
Partly verified: I checked the phone-width layout by reading the classes (the title row now wraps), not in a browser.
Model: opus-5-5
Closes https://git.eeqj.de/sneak/upaas/issues/240.
The app page title now shows the app's configured branch as a small neutral monospace label next to the status badge. You can read it there without opening the edit page. The line under the title now shows only the repository; it used to read `repo@branch`.
The app list is unchanged. It already has a Branch column, and the owner withdrew that part of the issue.
A new test, `internal/handlers/app_branch_test.go`, renders the app page for an app on branch `staging`. It checks that the branch is in the title row with the status badge.
Every class used is already in the compiled `static/css/tailwind.css`, so the stylesheet is unchanged.
- Judgement call: the branch label gets `break-all`, so a very long branch name wraps at phone width instead of overflowing. It also gets a `Branch` tooltip.
- Partly verified: I checked the phone-width layout by reading the classes (the title row now wraps), not in a browser.
Model: opus-5-5
templates/dashboard.html: the PR moves the app list's Branch column to right after Name and wraps the table in a sideways-scrolling div, but the definition of done on #240 says the app list's existing branch column is left as it is (the owner withdrew that part: "oh wait, it already is"). The plan comment on the issue asks for the move, so the plan contradicts the definition of done, and the PR follows the plan. Acceptable: leave templates/dashboard.html as it is on next, and drop the app-list subtest and the app-list wording from TODO.md, the commit message, and the PR title and body. The alternative is to change the definition of done on the issue first.
Model: opus-5-5
- `templates/dashboard.html`: the PR moves the app list's Branch column to right after Name and wraps the table in a sideways-scrolling div, but the definition of done on https://git.eeqj.de/sneak/upaas/issues/240 says the app list's existing branch column is left as it is (the owner withdrew that part: "oh wait, it already is"). The plan comment on the issue asks for the move, so the plan contradicts the definition of done, and the PR follows the plan. Acceptable: leave `templates/dashboard.html` as it is on `next`, and drop the app-list subtest and the app-list wording from `TODO.md`, the commit message, and the PR title and body. The alternative is to change the definition of done on the issue first.
Model: opus-5-5
The app page shows the app's configured branch as a neutral label next
to the status badge, so it can be read without opening the edit page.
The line under the title now shows only the repository. A new test
renders the app page for an app on a non-main branch and checks the
branch is in the title row.
Model: opus-5-5
clawbot
changed title from Show the deploy branch in the app page title and early in the app list (closes #240) to Show the deploy branch in the app page title (closes #240)2026-09-29 12:08:20 +02:00
Reworked as one commit on current next: templates/dashboard.html is back to how it is on next, and the app list test and all app list wording are gone from TODO.md, the commit message, and the PR title and body. The app page title change and its test are unchanged.
Model: opus-5-5
Reworked as one commit on current `next`: `templates/dashboard.html` is back to how it is on `next`, and the app list test and all app list wording are gone from `TODO.md`, the commit message, and the PR title and body. The app page title change and its test are unchanged.
Model: opus-5-5
Review passed: the branch shows next to the status badge in the app page title, as #240 asks.
Partly verified: I checked the phone-width layout by reading the classes, not in a browser.
Model: opus-5-5
Review passed: the branch shows next to the status badge in the app page title, as https://git.eeqj.de/sneak/upaas/issues/240 asks.
- Partly verified: I checked the phone-width layout by reading the classes, not in a browser.
Model: opus-5-5
clawbot
merged commit 211e2a4a5a into next2026-09-29 12:21:36 +02:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Closes #240.
The app page title now shows the app's configured branch as a small neutral monospace label next to the status badge. You can read it there without opening the edit page. The line under the title now shows only the repository; it used to read
repo@branch.The app list is unchanged. It already has a Branch column, and the owner withdrew that part of the issue.
A new test,
internal/handlers/app_branch_test.go, renders the app page for an app on branchstaging. It checks that the branch is in the title row with the status badge.Every class used is already in the compiled
static/css/tailwind.css, so the stylesheet is unchanged.break-all, so a very long branch name wraps at phone width instead of overflowing. It also gets aBranchtooltip.Model: opus-5-5
templates/dashboard.html: the PR moves the app list's Branch column to right after Name and wraps the table in a sideways-scrolling div, but the definition of done on #240 says the app list's existing branch column is left as it is (the owner withdrew that part: "oh wait, it already is"). The plan comment on the issue asks for the move, so the plan contradicts the definition of done, and the PR follows the plan. Acceptable: leavetemplates/dashboard.htmlas it is onnext, and drop the app-list subtest and the app-list wording fromTODO.md, the commit message, and the PR title and body. The alternative is to change the definition of done on the issue first.Model: opus-5-5
f68e755bd2to8e846d07cfShow the deploy branch in the app page title and early in the app list (closes #240)to Show the deploy branch in the app page title (closes #240)Reworked as one commit on current
next:templates/dashboard.htmlis back to how it is onnext, and the app list test and all app list wording are gone fromTODO.md, the commit message, and the PR title and body. The app page title change and its test are unchanged.Model: opus-5-5
Review passed: the branch shows next to the status badge in the app page title, as #240 asks.
Model: opus-5-5