Skip to content

feat(tray): add an attributed "Support ActivityWatch" menu item - #147

Open
TimeToBuildBob wants to merge 1 commit into
ActivityWatch:masterfrom
TimeToBuildBob:feat/tray-support-item
Open

TimeToBuildBob wants to merge 1 commit into
ActivityWatch:masterfrom
TimeToBuildBob:feat/tray-support-item

Conversation

@TimeToBuildBob

Copy link
Copy Markdown
Contributor

Adds a static Support ActivityWatch item under Open API Browser in the tray menu. It opens https://activitywatch.net/go/?src=qt-tray through the existing open_url path, so Linux keeps the AppImage-safe xdg-open environment. It is the aw-qt twin of ActivityWatch/aw-tauri#303. There is no nag and no toggle: it is one menu entry.

qt-tray is registered as a /go/ source in ActivityWatch/activitywatch.github.io#100. Until that merges, the click still lands on the subscribe page but counts as other.

Tests: new tests/test_support_link.py asserts the exact URL. It passes along with test_opener_env.py.

Not verified: the rendered tray menu. I had no live Qt/system-tray session, so there are no screenshots.

Git-Session-Id: 7afbde1c-79ba-4bb6-8cdf-f5cc3457fda4
@greptile-apps

greptile-apps Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge.

Summary

Adds a static “Support ActivityWatch” tray entry below “Open API Browser”.

  • The tray menu now has a Support ActivityWatch link.

Reviews (1) · Last reviewed commit: "feat(tray): add an attributed "Support A..." · Reviewed by Greptile

@TimeToBuildBob

Copy link
Copy Markdown
Contributor Author

All six checks pass on 50d4b29 (Linux, macOS, Windows, CodeQL and Greptile); Greptile is 5/5 with no inline findings, and the branch is mergeable. No code changes needed. Waiting for maintainer review/merge; the qt-tray attribution registration remains tracked separately in ActivityWatch/activitywatch.github.io#100.

@TimeToBuildBob

TimeToBuildBob commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor Author

🤖 AI code review

Adds SUPPORT_URL and open_support() in aw_qt/trayicon.py, with the helper passing https://activitywatch.net/go/?src=qt-tray to the existing open_url() function. Adds a "Support ActivityWatch" tray action below "Open API Browser" and a test asserting the URL passed by open_support().

Safe to merge — no P0/P1 findings

Confidence 5/5

✅ No findings. The diff looks correct to me on this pass.

Files changed (2) — the diff as I read it
  • aw_qt/trayicon.py — Adds SUPPORT_URL, open_support(), and a "Support ActivityWatch" action in TrayIcon._build_rootmenu().
  • tests/test_support_link.py — Adds a test that replaces trayicon.open_url with a recorder and asserts that open_support() passes the exact attributed URL.

Reviewed 50d4b29e815a · panel: codex:gpt-6.1-sol · subscription engine · 11s · about this reviewer

Maintainer commands

@TimeToBuildBob review (own line) — fresh review · @TimeToBuildBob fix — a worker acts on the findings. Once per comment; 👀 = received.

Outage coverage: 1-pass, not merge evidence. Subscription review supplied findings while full consensus capacity was unavailable. The consensus merge gate remains unsatisfied.

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