🐛 Bug Report: IPopoverNotificationCenterProps missing header arguments

[david-morris]

📜 Description

The types for the PopoverNotificationCenter component imply that it's not possible to let the header contain the button to edit the notification settings in the popover component. This is not true; I can use // @ts-expect-error as a workaround.

👟 Reproduction steps

1. Use typescript
2. use PopoverNotificationCenter
3. supply a header render prop that contains a button to open notification settings

👍 Expected behavior

The header types should match INotificationCenterProps

👎 Actual Behavior with Screenshots

The header types

Novu version

Novu SaaS

npm version

NA

node version

NA

📃 Provide any additional context for the Bug.

I also filed #3285 and am still using this render prop.
I'm updating from the client version 0.3 to 2.0.

👀 Have you spent some time to check if this bug has been raised before?

• I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to submit PR?

None

[linear]

NV-5171 🐛 Bug Report: IPopoverNotificationCenterProps missing header arguments

[SokratisVidros]

Hey @david-morris! Thanks for reporting this.

We've recently released Novu , that will supersede the @novu/notification-center package. Would you consider migrating to Inbox instead? I am also sharing the official migration guide.