Novu•4y ago

TypeError Cannot read properties of null (reading 'activeTabStoreId')

When using the

PopoverNotificationCenter

PopoverNotificationCenter

v0.9.2, I'm getting a

TypeError: Cannot read properties of null (reading 'activeTabStoreId')

TypeError: Cannot read properties of null (reading 'activeTabStoreId')

error when loading. I am self hosting it locally on docker and have set up the props on the Provider. I'm about to test it against the hosted version

DavidSouthmountain•11/25/22, 9:27 AM

How is you code with PopoverNotificationCenter looking?

dr.reallyOP•11/25/22, 9:28 AM

Manually set the unseenCount when I was trying to determine the issue

dr.reallyOP•11/25/22, 9:28 AM

note: I am using Remix, but I got it to work on my side project the other day

Ddr.really note: I am using Remix, but I got it to work on my side project the other day

DavidSouthmountain•11/25/22, 9:29 AM

yeah Remix is not the problem here I think

DavidSouthmountain•11/25/22, 9:29 AM

Happy to see more people that using Remix

dr.reallyOP•11/25/22, 9:29 AM

so yeah, even with the hosted version I'm still getting that activeTabeStoreId issue

DavidSouthmountain•11/25/22, 9:29 AM

What happens if you remove the tabs prop?

DDavidSouthmountain Happy to see more people that using Remix 🙂

dr.reallyOP•11/25/22, 9:29 AM

I love Remix

Ddr.really I love Remix

DavidSouthmountain•11/25/22, 9:29 AM

Me too

dr.reallyOP•11/25/22, 9:30 AM

DavidSouthmountain•11/25/22, 9:32 AM

ah maybe you need to do it like this:

<NovuProvider
  stores={[
    { storeId: 'activity', query: { feedIdentifier: 'activity' } },
  ]}
....

<NovuProvider
  stores={[
    { storeId: 'activity', query: { feedIdentifier: 'activity' } },
  ]}
....

DavidSouthmountain•11/25/22, 9:32 AM

found it in our docs here: https://docs.novu.co/notification-center/react-components/#defining-a-stores

React Component | Novu

Novu provides the @novu/notification-center a react library that helps to add a fully functioning notification center to your web application in minutes. Let's do a quick recap on how we can easily use it in your application:

dr.reallyOP•11/25/22, 9:32 AM

looking now, one moment

Novu_Bot•11/25/22, 9:32 AM

@dr.really, you just advanced to level 5!

DDavidSouthmountain ah maybe you need to do it like this: ``` <NovuProvider stores={[ { storeI...

dr.reallyOP•11/25/22, 9:34 AM

alright tried that on hosted, didn't work, let me try local

dr.reallyOP•11/25/22, 9:35 AM

the most recent version of the notifications center is v0.9.2 right? don't want to over look something lol

DavidSouthmountain•11/25/22, 9:35 AM

yes it is latest

dr.reallyOP•11/25/22, 9:36 AM

hmm nope getting the same error

DavidSouthmountain•11/25/22, 9:36 AM

DavidSouthmountain•11/25/22, 9:37 AM

and with out any tabs it is working?

dr.reallyOP•11/25/22, 9:37 AM

let me verify with no tabs

DavidSouthmountain•11/25/22, 9:37 AM

So it is working as basic as possible

dr.reallyOP•11/25/22, 9:38 AM

it's the same for me atleast thinking_pepe

DavidSouthmountain•11/25/22, 9:38 AM

hm then it is something else...

dr.reallyOP•11/25/22, 9:38 AM

not working either way I go about it..

DavidSouthmountain•11/25/22, 9:38 AM

and you removed both tabs and stores?

dr.reallyOP•11/25/22, 9:38 AM

let me try to turn it off and back on

and yessir removed both tabs and stores

DavidSouthmountain•11/25/22, 9:38 AM

dr.reallyOP•11/25/22, 9:39 AM

nope that wasn't it either

DavidSouthmountain•11/25/22, 9:40 AM

okay do you still get the same error?

dr.reallyOP•11/25/22, 9:40 AM

DavidSouthmountain•11/25/22, 9:41 AM

okay lets go dramatic

could you change to version 0.8.0?

dr.reallyOP•11/25/22, 9:41 AM

haha yeah one moment!

DavidSouthmountain•11/25/22, 9:41 AM

and try against the hosted version

dr.reallyOP•11/25/22, 9:42 AM

@novu/node

@novu/node

to 0.8.0 as well?

DavidSouthmountain•11/25/22, 9:42 AM

should not be needed

dr.reallyOP•11/25/22, 9:42 AM

installing

DavidSouthmountain•11/25/22, 9:42 AM

nice

dr.reallyOP•11/25/22, 9:43 AM

done, running now against hosted

--•11/25/22, 9:43 AM

@davidsouthmountain in the store we set the context as

null

null

by default so until the feed context is set again this could happen. Maybe we should set the default to empty object or do shortcut fallbacks to

{}

{}

everywhere

useFeedContext

useFeedContext

is used.
https://github.com/novuhq/novu/blob/2d3050a13b5dfc77b0d66e34ed2eac8a25299631/packages/notification-center/src/store/feed.context.ts#L10

GitHub

novu/feed.context.ts at 2d3050a13b5dfc77b0d66e34ed2eac8a25299631 · ...

The Open-Source Notifications Platform. Add a working notification center to your app in minutes.

novu/feed.context.ts at 2d3050a13b5dfc77b0d66e34ed2eac8a25299631 · novuhq/novu

---@davidsouthmountain in the store we set the context as `null` by default so unti...

DavidSouthmountain•11/25/22, 9:43 AM

yeah great find

--•11/25/22, 9:44 AM

@dr.really mind to open an issue with this? It can be a quick fix or a good Community ticket.

dr.reallyOP•11/25/22, 9:45 AM

Will do, let me finish this and I will open one up. Once my other PR is closed i'd be more than happy to toss one in. ( I messed my local next branch up building that outlook365 provider vs doing it on a seperate branch lol)

dr.reallyOP•11/25/22, 9:47 AM

alright, going back to 0.8.0 the bell showed up, but upon clicking on it, it states its not wrapped in a provider, so i'm testing something with mantine really quick to test a theory

DavidSouthmountain•11/25/22, 9:48 AM

and you are sure it is wrapped with the Novu provider?

dr.reallyOP•11/25/22, 9:52 AM

so initially i had it wrapped around Remix'

<Outlet />

<Outlet />

but displaying the icon in the Header section of Mantine's

AppShell

AppShell

component

dr.reallyOP•11/25/22, 9:52 AM

and that's what it was

dr.reallyOP•11/25/22, 9:52 AM

alright, moving back to 0.9.2 and testing that again, sorry for wasting your time lol

DavidSouthmountain•11/25/22, 9:53 AM

no problem great that we found the problem

dr.reallyOP•11/25/22, 9:53 AM

aye, lessons learned lol about mantine espically lol

TypeError Cannot read properties of null (reading 'activeTabStoreId')

Similar Threads

Similar Threads

Similar Threads