Skip to content
Snippets Groups Projects

Add Hilt and inject new MailboxNotificationManager

Merged Torsten Grote requested to merge 13-hilt into main
All threads resolved!

This also adds support for saving ViewModel state.

Closes #13 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Sebastian
  • Sebastian
  • I tested this on an API 16 device, it works

  • Sebastian
  • Sebastian
  • Sebastian
  • I'm still a bit overwhelmed with all the things looking a bit unfamiliar in Kotlin, so I fear I'm not yet very good at spotting certain details yet. Still, this looks good to me!

  • Torsten Grote added 1 commit

    added 1 commit

    • 94ce89b3 - Introduce string resource for notification content

    Compare with previous version

  • Sebastian resolved all threads

    resolved all threads

  • Sebastian approved this merge request

    approved this merge request

  • merged

  • Sebastian mentioned in commit 3973b619

    mentioned in commit 3973b619

  • Please register or sign in to reply
    Loading