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.

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

[✨ FEATURE]: create a notifications view #546

Open
marcgc21 opened this issue Jan 19, 2025 · 7 comments
Open

[✨ FEATURE]: create a notifications view #546

marcgc21 opened this issue Jan 19, 2025 · 7 comments
Assignees
Labels
Enhancement New feature or request Good first issue Good for newcomers good for wannabe gsocers Issues for the wannabe contributors to get to know the code better

Comments

@marcgc21
Copy link
Member

Feature Description πŸ“

We would like to have a view where the user can see the different notifications that it has received.

Motivation 🌟

Right now it is not possible to see the complete list of notifications or past notifications.

Expected Behavior πŸ€”

I would like to see a page with the different notifications. We must discuss which fields must appear on that view.

Additional Information ℹ️

No response

@marcgc21 marcgc21 added Enhancement New feature or request Good first issue Good for newcomers good for wannabe gsocers Issues for the wannabe contributors to get to know the code better labels Jan 19, 2025
@saltykheera
Copy link
Contributor

working on this issue Assign this to me

@saltykheera
Copy link
Contributor

screen-recorder-mon-jan-20-2025-02-35-25.webm

**Almost done just new sections for all past Notification left to make **
will make the PR on 7:30 AM at Spain time

@marcgc21
Copy link
Member Author

I think that as an starting point it will be nice if notifications stops being removed.
If we think in something like Instagram or Facebook, it is important to mantain the list of notifications received because it might be important at some point to recover and read it again.

I beliave that it will imply to think a little bit if the database modal do not suit to that functionality. What do you think @saltykheera make it sense for you?

@saltykheera
Copy link
Contributor

@marcgc21
in this pr i have taken care of that
indox will contain all the notification and will not get removed
whereas
unread will consists of notification that are not read once read it will get cleard but saved in indox

@saltykheera
Copy link
Contributor

notification.page.mp4

@marcgc21 have a look at this functionality in visual i have implemented in pr #553

@marcgc21
Copy link
Member Author

Wow, it looks very promising for a first version! I'll have to test it with my user since I have a lot of messages and it will be a good test of use.

@eulerbutcooler
Copy link

Can i still take up this issue? @marcgc21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Good first issue Good for newcomers good for wannabe gsocers Issues for the wannabe contributors to get to know the code better
Projects
None yet
Development

No branches or pull requests

3 participants