N
Novu•2y ago
empe

Required - Notification Group

What does this mean?
12 Replies
Paweł T.
Paweł T.•2y ago
@emilpearce it means that you have to specify the Notification Group 😛
Pawan Jain
Pawan Jain•2y ago
@emilpearce Did you find option to add group?
Prosper
Prosper•2y ago
We might need to rethink this flow. Many users, especially new will have no idea what a notification group is, and at this stage I don't think it should a requirement for them
Prosper
Prosper•2y ago
I had to click on "Settings" to have an idea where this error was hiding
Prosper
Prosper•2y ago
Prosper
Prosper•2y ago
If I go in to update the In-App notification template content, I get the error
Prosper
Prosper•2y ago
Prosper
Prosper•2y ago
@galiainouz @paweltymczuk It keeps me or anyone wondering what's happening on this particular page because everything has been done right
Gali Baum
Gali Baum•2y ago
I managed to reproduce. It should be solved after Pawel's work in this PR https://github.com/novuhq/novu/pull/3458
GitHub
[NV-2360] fix(web): fixed the issue when visiting the same workflow...
What change does this PR introduce? This PR fixes the issue that happens when visiting the template editor twice for the same template. Currently, the form state is not "set", which resul...
Gali Baum
Gali Baum•2y ago
regarding the flow - I feel its a very good point. Tagging @omerbenami @nikolayknyshov
Paweł T.
Paweł T.•2y ago
thanks Gali, I feel like all the errors handling and validation should be improved a little 🙂
Want results from more Discord servers?
Add your server