Novu•4y ago

Deleting Notifications from the In-App Notification Center

I'm developing a website for a client and we're using the in-app notification center using React. He was asking about allowing users to delete or clear notifications after they've been read. Currently we can set them to 'read' status but not delete them.

I looked through the docs, and briefly looked through the codebase and didn't see anything about deleting in app notifications. Is there currently a way I didn't see?

We're using the cloud hosted version if that matters.

bigheartedkbOP•9/29/22, 6:24 PM

We're using a custom Item renderer so technically I could return an empty div when the notification has been read. Which fixes it 'visually' but the data is still being passed down to the website.

Novu_Bot•9/29/22, 6:24 PM

@bigheartedkb, you just advanced to level 2!

Gali Baum•10/2/22, 8:15 AM

We definitely have plans to add delete(among other options) for a notification item

Gali Baum•10/2/22, 8:16 AM

I have a suggestion that might help, just for now. You could use the stores - which are a way to query notifications for a parameter (like feedIdentifier and also seen state of notification) @bigheartedkb

Gali Baum•10/2/22, 8:17 AM

 <NovuProvider
        stores={[{ storeId: 'default_store', query: { seen: false } }]}
        backendUrl={API_ROOT}
        socketUrl={WS_URL}
        subscriberId={user?._id as string}
        applicationIdentifier={environment?.identifier as string}
      >
        <PopoverWrapper />
      </NovuProvider>

 <NovuProvider
        stores={[{ storeId: 'default_store', query: { seen: false } }]}
        backendUrl={API_ROOT}
        socketUrl={WS_URL}
        subscriberId={user?._id as string}
        applicationIdentifier={environment?.identifier as string}
      >
        <PopoverWrapper />
      </NovuProvider>

Gali Baum•10/2/22, 8:18 AM

That will query only the notifications that were not read yet

Gali Baum•10/2/22, 8:19 AM

and then

useNotifications(storeId)

useNotifications(storeId)

empe•10/2/22, 9:42 AM

@bigheartedkb my mannnn!!!!!

Dima Grossman•10/2/22, 10:18 AM

@davidsouthmountain I think you have added already the remove message enedpoint right?

DavidSouthmountain•10/2/22, 10:19 AM

Yes I did and it can be found here https://github.com/novuhq/novu/blob/main/apps/api/src/app/messages/messages.controller.ts#L75

DavidSouthmountain•10/2/22, 10:20 AM

@dimagrossman maybe we should add the Messages api tag to be shown in swagger json so it is shown in api doc?

DavidSouthmountain•10/2/22, 10:23 AM

https://github.com/novuhq/novu/pull/1360

GitHub

fix: so messages endpoints are shown in swagger json by davidsoderb...

Dima Grossman•10/2/22, 10:24 AM

Oh yes, looks like a mistake. We should add a Tag.

Dima Grossman•10/2/22, 10:25 AM

@florian_lefebvre mind adding a github issue for this? !issue

Dima Grossman•10/2/22, 10:25 AM

!issue

Novu_Bot•10/2/22, 10:25 AM

Want to report a bug or request a feature?
Let's open an issue. Just click here

https://github.com/novuhq/novu/issues/new/choose

DavidSouthmountain•10/2/22, 10:25 AM

@dimagrossman

DavidSouthmountain•10/2/22, 10:25 AM

https://github.com/novuhq/novu/pull/1360

GitHub

fix: so messages endpoints are shown in swagger json by davidsoderb...

DavidSouthmountain•10/2/22, 10:25 AM

this solves that problem

Novu_Bot•10/2/22, 10:25 AM

@florian_lefebvre, you just advanced to level 4!

DavidSouthmountain•10/5/22, 4:30 AM

you could use refetch that useNotification expose?

DavidSouthmountain•10/5/22, 4:56 AM

@arc.13 could you see if this works now as well?

DavidSouthmountain•10/5/22, 4:57 AM

okay so lets see if we can solve this as well

DavidSouthmountain•10/5/22, 4:57 AM

Can I close the other one?

DavidSouthmountain•10/5/22, 4:57 AM

nice then I close both of them

Dima Grossman•11/28/22, 6:20 AM

@george.d do you recall by chance if we also expose unreadCount here?

DDima Grossman @george.d do you recall by chance if we also expose unreadCount here?

george.d•11/28/22, 6:51 AM

I don’t believe we do at the moment. The toggle between seen/read on the notifications center is still missing.

Dima Grossman•11/29/22, 7:30 AM

Do you mind creating an issue for that?

DDima Grossman Do you mind creating an issue for that?

george.d•11/29/22, 9:38 AM

Not sure if you meant me or someone else, but not problem at all i will create shortly

george.d•11/29/22, 10:08 AM

Hopefully it is well detailed

https://github.com/novuhq/novu/issues/2123

GitHub

[NV-1266] Notification-Center bell seen/read toggle · Issue #2123 ·...

At the current state notification-center bell counter uses the seen count in order to present the bell dot and counter on the notification center header. What we want is the ability for the user of...

<NovuProvider stores={[{ storeId: 'default_store', query: { seen: false } }]} backendUrl={API_ROOT} socketUrl={WS_URL} subscriberId={user?._id as string} applicationIdentifier={environment?.identifier as string} > <PopoverWrapper /> </NovuProvider>

Deleting Notifications from the In-App Notification Center

Similar Threads

Similar Threads

Similar Threads