fix(notify): accept canonical shared config schema - #246
Conversation
Git-Session-Id: b686
Greptile SummaryThe PR extends Android’s notification-setting parser to consume the canonical object and snake_case schema while retaining legacy array compatibility.
Confidence Score: 5/5The PR appears safe to merge with no actionable correctness or security issues identified. The canonical parser matches the documented setting shape, preserves intentional empty and fallback behavior, and retains compatibility with the legacy representation. Important Files Changed
Reviews (1): Last reviewed commit: "fix(notify): accept canonical shared con..." | Re-trigger Greptile |
|
CI-green and mergeable (Greptile 5/5) — waiting only on a maintainer click. This PR is ready to merge, but the bot has pull-only access to this repo and can't self-merge. Project monitoring marked it Automated merge handoff for
|
Summary
aw-notifysetting written by fix(settings): unify aw-notify config schema aw-webui#924category: "All"to Android's aggregate category and nullable labels to useful defaultsalertslist so users can disable alertsThis closes the Android parser slice of #201. Desktop consumption remains in ActivityWatch/aw-notify-rs#39.
Test plan
JAVA_HOME=/opt/android-studio/jbr ANDROID_HOME=/home/bob/Android/Sdk ./gradlew :mobile:testDebugUnitTest --no-daemon --console=plain