Notification bell

[DigitalSeneca]

fixes #43

Description of changes

• Show number of unread notifications
  • Calculated when app is open
  • Recalculated when opening NotificationPage and waiting 5 seconds (to let it load the notifications from the api)
  • Recalculated when opening a notification (so it is marked as read)

Screenshot	GIF

[epou]

Hey! Does the counter badge "break" visually when the count reaches two digits?

[DigitalSeneca]

@epou

Hey! Does the counter badge "break" visually when the count reaches two digits?

Seems to be working fine visually with 2 digits. Screenshot:

[epou]

Cool! Can we fetch the data from the API in the main_vc.dart asynchronously and pass the list into the NotificationsPage view? This API call is quite slow and would be nice to be called every X minutes in the background and on each app start to avoid waiting every time the user goes to the notification page.

[DigitalSeneca]

Cool! Can we fetch the data from the API in the main_vc.dart asynchronously and pass the list into the NotificationsPage view? This API call is quite slow and would be nice to be called every X minutes in the background and on each app start to avoid waiting every time the user goes to the notification page.

yes, I'll do that. The minimum amount of time between periodic background tasks is every 15min for Android, is that good or should I do a different interval?

[epou]

I didn't know about the 15min constraint for background tasks... Maybe it's too much. In that case, you can keep it as is.

By the way, I've seen there is a native class for badges in material. It would be worth a try:

• https://api.flutter.dev/flutter/material/Badge-class.html
• https://m3.material.io/components/badges/overview

[epou]

Looks good to me! I feel curious if this https://api.flutter.dev/flutter/material/Badge-class.html might help in making the code easier, I feel the native library might consider a broad number of case scenarios and less prone to bugs.

Just to be consistent with the linked issue (#43), I'm not sure if this is working or not:

The app logo have also a badge number with the unread notifications

I've found this, which might be helpful: https://stackoverflow.com/questions/61979970/how-to-show-a-notification-badge-on-the-app-icon-using-flutter

[DigitalSeneca]

I'm sorry I didn't address the comment about the badge before requesting a new review. I'll pay more attention next time before asking for a new review to not waste your time

The app logo have also a badge number with the unread notifications

This is not yet implemented, only the badge inside of the app. I've tried it and I could not make it work for now, isAppBadgeSupported() returns true but nothing shows app in the icon

Here's the screenshot after the changes. I've also increased the bell size for it to be similar to the adn image next to it

with unread	without unread	10+ view