Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Add a notification preview / test screen [1 day] #1235

Closed
pfent opened this issue Dec 6, 2018 · 3 comments · Fixed by #1562
Closed

Add a notification preview / test screen [1 day] #1235

pfent opened this issue Dec 6, 2018 · 3 comments · Fixed by #1562

Comments

@pfent
Copy link
Contributor

pfent commented Dec 6, 2018

Follow up from previous issues; #879, #1162, #1165

Even though we really tried to fix this issue, we're still getting occasional crashes in NotificationScheduler.
Also the current way of throttling Notifications isn't exactly nice or precise 😞.

As a first step to tackle this, I'd like to introduce a notification preview screen (probably buried somewhere in the settings), which displays all scheduled notifications.

Added benefit:

  • Users see all notifications we have currently scheduled for them
  • We can give users more control over the notifications, maybe even allow them to adjust the frequency we schedule some notifications
  • Add a possibility to debug notifications and enhance the intelligence part of notification scheduling
@kordianbruck
Copy link
Member

Good idea, this would also allow for devs an easier access to check if their coded notification actually works...

@thellmund
Copy link
Contributor

Great idea to make debugging easier. I don’t think it will be of interest to end users, but if we bury it “somewhere in the settings”, that should be fine.

@jacqueline8711 jacqueline8711 changed the title Add a notification preview / test screen Add a notification preview / test screen [1 day] Feb 26, 2019
@jacqueline8711 jacqueline8711 added this to the SS 2019 milestone Feb 26, 2019
@kunzef
Copy link
Contributor

kunzef commented Apr 16, 2023

Hey, just wanted to know if the preview is still needed? If so I would like to try to implement it.

CommanderStorm added a commit that referenced this issue May 27, 2023
* Added notification overview

* Added queues to retrive scheduled and active notifications
Adjusted NotificationDebugViewFragment to display these

* Fixed falsely added whitespace in DrawerMenuHelper.kt

* Added logic in InformationActivity.kt for opening the NotificationOverview after 5 quick clicks

* Changed to Activity and improved file structure

* Added localization to StickyList headers

* Added padding to StickyListItem

* Added notifications from AlarmManager per NotificationType

* Added get PendingIntent for every ScheduledNotification

* Only textview txtVersion triggers click now

* Removed unnecessary import

* fixed a typo in the translations

* Fixed ktlint errors

* Fixed ktlint errors

---------

Co-authored-by: Frank Elsinga <[email protected]>
tobiasjungmann pushed a commit that referenced this issue Aug 4, 2023
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Replaced Anko in NotificationScheduler.kt

Replaced Anko in NotificationUtils.kt

Added preference key replaced Anko in BaseNavigationActivity.kt

Beautify by removing this

Replaced Anko ConnectivityManager in ProgressActivity.kt

Replaced Anko SearchManager in FragmentForSearching.kt

Replaced NotificationManager and SharedPreferences in SettingsFragment

Removed unnecessary comment

Replaced WifiManager in EduroamCard.kt

Replaced DefaultSharedPreferences and WifiManager in EduroamCard.kt

Replaced WifiManager in EduroamFixCardViewHolder.kt

Replaced DefaultSharedPreferences in TopNewsCard.kt

Replaced InputMethodManager in OnboardingStartFragment.kt

Replaced ConnectivityManager in MainFragment.kt

Replaced textColor in TicketAmountViewHolder.kt

Replaced AlarmManager in MVVWidget.kt

Replaced NotificationManager in FcmReceiverService.kt

Replaced WifiManager in ScanResultsAvailableReceiver.kt

Replaced ConnectivityManager in NetUtils.kt

Replaced DefaultSharedPreferences in Utils.kt

Replaced ConnectivityManager in BaseFragment.kt

Replaced DefaultSharedPreferences in OnboardingExtrasFragment.kt

Removed unused import in BaseFragment.kt

Replaced all android preference manager with android x
This solved the getDefaultSharedPreferenceDeprecated message

Replaced getSharedPreference with PreferenceManager default Preference

Calendar Month view (#1558)

* Provisional month view

* Deleted unnecessary import statement

* Update calendar_month_view.xml

* New approach for the month view

* improved the desingn

* Apply suggestions from code review

* Applyed formatting suggestions from code review

* renamed some functions

* implemented the swithcing behaviour via an enum instead of two bools

* renamed another function

* Clarifed that monday is the first day of the Week

* Apply suggestions from code review

Co-authored-by: Marius Wagner <[email protected]>

* formatting fixes

* formatting fix

---------

Co-authored-by: ge78fug <[email protected]>
Co-authored-by: Marius Wagner <[email protected]>

Implement notification preview mentioned in issue #1235 (#1562)

* Added notification overview

* Added queues to retrive scheduled and active notifications
Adjusted NotificationDebugViewFragment to display these

* Fixed falsely added whitespace in DrawerMenuHelper.kt

* Added logic in InformationActivity.kt for opening the NotificationOverview after 5 quick clicks

* Changed to Activity and improved file structure

* Added localization to StickyList headers

Removed unnecessary import

Fixed ktlint

Fixed ktlint formatting

Fixed ktlint formatting

Updated build.gradle:app
tobiasjungmann pushed a commit that referenced this issue Aug 4, 2023
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Replaced Anko in NotificationScheduler.kt

Replaced Anko in NotificationUtils.kt

Added preference key replaced Anko in BaseNavigationActivity.kt

Beautify by removing this

Replaced Anko ConnectivityManager in ProgressActivity.kt

Replaced Anko SearchManager in FragmentForSearching.kt

Replaced NotificationManager and SharedPreferences in SettingsFragment

Removed unnecessary comment

Replaced WifiManager in EduroamCard.kt

Replaced DefaultSharedPreferences and WifiManager in EduroamCard.kt

Replaced WifiManager in EduroamFixCardViewHolder.kt

Replaced DefaultSharedPreferences in TopNewsCard.kt

Replaced InputMethodManager in OnboardingStartFragment.kt

Replaced ConnectivityManager in MainFragment.kt

Replaced textColor in TicketAmountViewHolder.kt

Replaced AlarmManager in MVVWidget.kt

Replaced NotificationManager in FcmReceiverService.kt

Replaced WifiManager in ScanResultsAvailableReceiver.kt

Replaced ConnectivityManager in NetUtils.kt

Replaced DefaultSharedPreferences in Utils.kt

Replaced ConnectivityManager in BaseFragment.kt

Replaced DefaultSharedPreferences in OnboardingExtrasFragment.kt

Removed unused import in BaseFragment.kt

Replaced all android preference manager with android x
This solved the getDefaultSharedPreferenceDeprecated message

Replaced getSharedPreference with PreferenceManager default Preference

Calendar Month view (#1558)

* Provisional month view

* Deleted unnecessary import statement

* Update calendar_month_view.xml

* New approach for the month view

* improved the desingn

* Apply suggestions from code review

* Applyed formatting suggestions from code review

* renamed some functions

* implemented the swithcing behaviour via an enum instead of two bools

* renamed another function

* Clarifed that monday is the first day of the Week

* Apply suggestions from code review

Co-authored-by: Marius Wagner <[email protected]>

* formatting fixes

* formatting fix

---------

Co-authored-by: ge78fug <[email protected]>
Co-authored-by: Marius Wagner <[email protected]>

Implement notification preview mentioned in issue #1235 (#1562)

* Added notification overview

* Added queues to retrive scheduled and active notifications
Adjusted NotificationDebugViewFragment to display these

* Fixed falsely added whitespace in DrawerMenuHelper.kt

* Added logic in InformationActivity.kt for opening the NotificationOverview after 5 quick clicks

* Changed to Activity and improved file structure

* Added localization to StickyList headers

Removed unnecessary import

Fixed ktlint

Fixed ktlint formatting

Fixed ktlint formatting

Updated build.gradle:app
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants