search

uchicago-cs / chigame

BSD 3-Clause "New" or "Revised" License
2 stars 0 forks source link

Add Junk box for deleted Notifications #339

Closed AndrewM131 closed 11 months ago

AndrewM131 commented 11 months ago

Currently you can delete notifications on the admin page. They're marked as deleted but they're not actually removed (visible attribute is set to false). Add a filtering feature where the visibility is set to false.

henrietta-k commented 11 months ago

Update: had a group meeting on Nov 30 to discuss how to implement this. Will add a filtering option to allow the user to only view removed (visible attribute set to false) notifications

henrietta-k commented 11 months ago

Update Nov 30: a button was added to the user's inbox called "Show Deleted Notifications" so that when the user clicks on it, it'll take them to a new page with all their deleted notifications. All the notifications on this page can be moved back to the original inbox if the user clicks on "Move to Inbox," and they will all be marked as unread.

henrietta-k commented 11 months ago

Closing statement: The issue has been addressed. In the user's inbox, they can now choose to see all their deleted notifications and move it back into their main inbox (marked as unread) if they want. All the assignees worked together to write the code but I was in charge of making the GitHub PR.

elizabethli31 commented 10 months ago

Issue Score: Excellent

Comments: Cool addition to the inbox functionality!