knossos icon indicating copy to clipboard operation
knossos copied to clipboard

Collapsible grouped notifications

Open devBoi76 opened this issue 3 years ago • 1 comments

image

This PR adds the long awaited grouped notifications. When there are multiple update notifications of the same project, they are automatically grouped in order to not clutter the notifications page. If more than three are present, a "More" button is shown.

Note: currently notifications are grouped based on their title, as that is the only unique identifier in the notification API. Same thing with the version. In the future it might be a good idea to include those fields in the API.

devBoi76 avatar Sep 19 '22 18:09 devBoi76

Hold on, I accidentally left in a console.log. I'll remove it real quick

devBoi76 avatar Sep 19 '22 19:09 devBoi76

Oops, sorry, I missclicked on my phone. Didn't mean to change those :|

devBoi76 avatar Oct 21 '22 21:10 devBoi76

Please resolve conflicts!

triphora avatar Nov 13 '22 15:11 triphora

I'll resolve the conflicts some time later this week, probably tomorrow or on Friday

devBoi76 avatar Nov 16 '22 12:11 devBoi76

Do not merge yet, I'll still make a few changes

devBoi76 avatar Nov 17 '22 18:11 devBoi76

This PR is opened half a year ago. Last commit is three months ago. I still waiting for this feature.

I can help with developing. I know Vue and Nuxt and I'm a frontend developer. Do you need some help?

Grawl avatar Apr 15 '23 08:04 Grawl

We're going to add this feature, but we are going to overhaul how notifications work on the backend first. This feature should come to the site soon!

Prospector avatar Apr 15 '23 19:04 Prospector