Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Restore notifications when app starts #1509

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from
Open

Restore notifications when app starts #1509

wants to merge 12 commits into from

Commits on Sep 17, 2022

  1. Remove habits from active list when should not be shown

    @felixwiemuth
    felixwiemuth committed Sep 17, 2022
    Configuration menu
    Copy the full SHA
    33100df View commit details
    Browse the repository at this point in the history

  2. Remove redundant rescheduling after reboot 

    The action is already performed via HabitsApplication.onCreate.
It is sufficient that there is a boot receiver.
    @felixwiemuth
    felixwiemuth committed Sep 17, 2022
    Configuration menu
    Copy the full SHA
    0d1d8a9 View commit details
    Browse the repository at this point in the history

  3. Add dependency and plugin: kotlinx-serialization-json

    @felixwiemuth
    felixwiemuth committed Sep 17, 2022
    Configuration menu
    Copy the full SHA
    794289b View commit details
    Browse the repository at this point in the history

  4. Persist notifications 

    - Persist the map of active notifications on every change and reload it on
application startup.
- Reshow all active notifications on application startup using the original
reminder due time.
    @felixwiemuth
    felixwiemuth committed Sep 17, 2022
    Configuration menu
    Copy the full SHA
    09c46f5 View commit details
    Browse the repository at this point in the history

  5. Show previously shown notifications silently

    @felixwiemuth
    felixwiemuth committed Sep 17, 2022
    Configuration menu
    Copy the full SHA
    857e21a View commit details
    Browse the repository at this point in the history

  6. Add app update broadcast receiver 

    So that notifications are also restored after app upgrade
    @felixwiemuth
    felixwiemuth committed Sep 17, 2022
    Configuration menu
    Copy the full SHA
    68f2639 View commit details
    Browse the repository at this point in the history

Commits on Sep 18, 2022

  1. Remove shown field from NotificationData 

    It is not necessary to keep this field as in the map of active
notifications (and its persisted form) this field is always `true`.
    @felixwiemuth
    felixwiemuth committed Sep 18, 2022
    Configuration menu
    Copy the full SHA
    4174359 View commit details
    Browse the repository at this point in the history

Commits on Sep 25, 2022

  1. Catch serialization exception 

    In case serialized notifications cannot be deserialized, continue with an empty map of active notifications
    @felixwiemuth
    felixwiemuth committed Sep 25, 2022
    Configuration menu
    Copy the full SHA
    f1c104a View commit details
    Browse the repository at this point in the history

  2. Better name: NotificationData.silent

    @felixwiemuth
    felixwiemuth committed Sep 25, 2022
    Configuration menu
    Copy the full SHA
    015c0d1 View commit details
    Browse the repository at this point in the history

Commits on Oct 2, 2022

  1. Add tests for active notifications persistence

    @felixwiemuth
    felixwiemuth committed Oct 2, 2022
    Configuration menu
    Copy the full SHA
    1540427 View commit details
    Browse the repository at this point in the history

  2. Make NotificationData data class

    @felixwiemuth
    felixwiemuth committed Oct 2, 2022
    Configuration menu
    Copy the full SHA
    e65357b View commit details
    Browse the repository at this point in the history

  3. Fix code style

    @felixwiemuth
    felixwiemuth committed Oct 2, 2022
    Configuration menu
    Copy the full SHA
    ee0663e View commit details
    Browse the repository at this point in the history