ActivityWatch / ActivityWatch/aw-android

aw-notify: settingsStore integration and shared webui config surface

Đang mở
#201 21 bình luận 0 reaction 1 người được giao Được @TimeToBuildBob nhận Xem trên GitHub
Ngôn ngữ chính
Kotlin
Star
265
Fork
57
Merge trung bình
2 ngày 21 giờ
Pull request đã merge (30 ngày)
19

Mô tả

Follow-up to #199 (merged) — centralising alert configuration so Android shares the same config surface as aw-notify on desktop.

## What's missing from the MVP

The `NotifyWorker` shipped in #199 hardcodes `DEFAULT_ALERTS` directly in Kotlin. The only server-side setting it reads is `startOfDay`. Everything else (categories, thresholds, enabled/disabled) requires a code change.

## Goals

1. **settingsStore integration** — store alert config under `/api/0/settings/aw-notify/alerts` (or a sub-key of the existing settings namespace). The Android app reads this at worker startup instead of the hardcoded list, matching the server-side config model aw-notify desktop already uses.

2. **Shared webui config surface** — once the config lives in the settings API, the aw-webui can expose a config panel identical to (or compatible with) the desktop aw-notify settings, so users manage thresholds in one place regardless of client.

3. **Reduced default thresholds** — the current defaults (`All` at 1h/2h/4h/6h/8h, `Work` at 15m/30m/1h/2h/4h, etc.) are too dense for most days. Trim to 2–3 thresholds per category max, or align with whatever the desktop aw-notify defaults converge to after #200 lands.

4. **Migration path** — if a user has no server-side config, fall back to a minimal built-in default (not the current verbose list) so first-run behaviour is conservative.

## Out of scope here

- Anomaly/percentile-based notifications → tracked in #200
- App-layer quiet-hours (the OS `aw_notify_channel` + DnD is sufficient for now)

## Cross-references

- #199 — the initial notify implementation (merged)
- #200 — smart anomaly-detection notifications (v2 direction)
- [aw-notify desktop](https://github.com/ActivityWatch/aw-notify) — the Python daemon this should share config with

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.