Skip to content

Conversation

@williscool
Copy link
Owner

@williscool williscool commented Jan 22, 2026

Added toIntentString() and fromIntentString() methods to EventAlertRecordKey class to ensure the event key format used in Intent extras is defined in one place. This prevents format mismatches between serialization and parsing.

  • EventAlertRecordKey.toIntentString(): Serializes key for Intent extras
  • EventAlertRecordKey.fromIntentString(): Parses key from Intent extras
  • Updated ActiveEventsFragment to use toIntentString()
  • Updated ApplicationController to use fromIntentString()

Note

Unifies the event key format used in Intent extras by making EventAlertRecordKey the single source of truth.

  • Adds toIntentString() and fromIntentString() to EventAlertRecordKey for Intent serialization/parsing
  • Replaces manual string building in ActiveEventsFragment with it.key.toIntentString() when launching SnoozeAllActivity
  • Replaces ad-hoc parsing in ApplicationController.snoozeSelectedEvents with EventAlertRecordKey.fromIntentString()

Written by Cursor Bugbot for commit 055c440. This will update automatically on new commits. Configure here.

Added toIntentString() and fromIntentString() methods to EventAlertRecordKey
class to ensure the event key format used in Intent extras is defined in
one place. This prevents format mismatches between serialization and parsing.

- EventAlertRecordKey.toIntentString(): Serializes key for Intent extras
- EventAlertRecordKey.fromIntentString(): Parses key from Intent extras
- Updated ActiveEventsFragment to use toIntentString()
- Updated ApplicationController to use fromIntentString()

Co-authored-by: wharris <wharris@upscalews.com>
@williscool williscool enabled auto-merge (squash) January 22, 2026 00:22
@williscool williscool disabled auto-merge January 22, 2026 00:27
@williscool williscool enabled auto-merge (squash) January 22, 2026 00:28
@github-actions
Copy link

Build artifacts for PR #231 (commit b880d36) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

@williscool williscool merged commit 3e62d99 into master Jan 22, 2026
9 of 14 checks passed
@github-actions
Copy link

Code Coverage Report

Overall Project 27.36% -0.03%
Files changed 78.95% 🍏

File Coverage
ApplicationController.kt 59.4% 🍏
EventAlertRecord.kt 59.02% -0.62% 🍏
ActiveEventsFragment.kt 49.92% -1.26%

@github-actions
Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 27.36
Changed Files 57.91

View detailed coverage report

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.

3 participants