Skip to content

Comments

Notif#36

Open
SVA-BL00 wants to merge 8 commits intomainfrom
notif
Open

Notif#36
SVA-BL00 wants to merge 8 commits intomainfrom
notif

Conversation

@SVA-BL00
Copy link
Owner

@SVA-BL00 SVA-BL00 commented Jun 14, 2024

Pull Request

Added styling to show list of notifications

Related Issues

For future developments:

  • Add a boolean within the user's profile. Read/Unread notifications. If unread, add a circle badge to notifications. When read, disable circle badge.
  • Send a mail to the user every week with all of the notifications. This implies that all notifs during a week must be saved in a database and after they're sent, they must be deleted.

Changes Made

  • Added style and click functionality for notif button.

Additional Comments

Due to time issues, the implementation of the full functionality of the notification button won't be possible. This implies changes to the database structure itself and will be a problem for the moment.

@SVA-BL00 SVA-BL00 linked an issue Jun 14, 2024 that may be closed by this pull request
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.

Add notification feature

3 participants