You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Mobile: show checks, review and watch state for linked pull requests, like desktop
#20212
Requesting agreement on direction and scope before I write any code.
Problem: When an agent watches pull requests, the phone shows only a "Waiting on N monitors" pill, which cannot be tapped. The linked pull requests list at the bottom of the Git sheet shows "open · Watching" for each one, with no checks or review state. On desktop, the thread's pull requests panel shows a checks glyph and a review decision glyph on each row, and its menu has Watch for changes / Stop watching. From a phone I cannot see which watched PR is failing or approved, or stop or start a watch.
Proposal: On each mobile linked pull request row, show the same checks and review decision state as desktop (snapshot.checksState, snapshot.reviewDecision), and add Watch / Stop watching, gated on capabilities.threadPullRequestWatch like the web panel. Optionally, tapping the monitors pill opens the thread's linked pull requests list.
Scope: Mobile only. It reuses the existing snapshot data and the existing watch commands. No server or contract changes.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
Requesting agreement on direction and scope before I write any code.
snapshot.checksState,snapshot.reviewDecision), and add Watch / Stop watching, gated oncapabilities.threadPullRequestWatchlike the web panel. Optionally, tapping the monitors pill opens the thread's linked pull requests list.LinkedPullRequestChainsrow after it merges, so the two do not conflict. feat(mobile): stop leftover background work from the status pill #18287 makes the status pill tappable to stop background work, so the pill part may fit there instead.Would you approve the row glyphs and the watch toggle as scope? Is the pill tap wanted, or should I leave it out?
All reactions