Skip to content

feat(statuses): show active mobile devices - #679

Open
dannash100 wants to merge 3 commits into
mainfrom
feat/mobile-usage
Open

dannash100 wants to merge 3 commits into
mainfrom
feat/mobile-usage

Conversation

@dannash100

@dannash100 dannash100 commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

A server's page doesn't show whether anyone uses the mobile app against it. Its public name only says it's listed for mobile.

mobile-page

🦸 Review Hero

  • Run Review Hero
  • Run Review Hero on Claude (before merge) Final review with Claude just before merging a substantive PR. Tick one or the other.

@dannash100

Copy link
Copy Markdown
Contributor Author

🤖 QA

  • Application page shows "Mobile devices (30d)", including 0 (e2e server-figures.spec.ts). Mutation-checked: hiding the field fails it.
  • No mobile field when no source reports one (e2e).
  • Status snapshot API returns the count, zero included (snapshot_reports_active_mobile_devices).

QA found the first push only showed it in the incident snapshot popup, which reads a different endpoint from the application page. Fixed in 3c45faa.

Not yet done: smoke against a real central once bestool#990 is released.

This branch has not been deployed

No deployments
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