Notification Banners

Theme component
21

Hey @gormus, I’m getting this error while trying to install:

An error occurred: Error creating upload asset: splide_css. Original filename Sorry, the file you are trying to upload is not authorized (authorized extensions: wasm, jpg, jpeg, png, woff, woff2, svg, eot, ttf, otf, gif, webp, avif, js).

Update: It worked, I actually forgot to add CSS in extensions.

3 Likes
22

I add two banners to the same outlet and select carousel on both. But they only show stacked. Am I missing something for the carousel to work?

1 Like
23

Hi @nolo there needs to be at least 2 banners in the same outlet for the carousel to work; and of course the Display in a Carousel option must be enabled on all banners too. The carousel won’t render if only one of the banners enabled.

Also;

  1. do you get any javascript errors in the browser console? Any of them reference to notification banners?
  2. condering there may be a conflict; were you able to isolate the issue to notification banners by disabling the other themes, components and plugins? or by using the safe mode?

Alternatively, would you mind sharing the settings? So, I can import and try it on my end…

  1. On the Discourse Notification Banners admin page, click on the Settings Editor` button at the bottom of the page.
  2. Copy the JSON data
  3. Strip any sensitive data you may have
  4. Share the JSON data in reply here, or DM me
2 Likes
24

Thanks @gormus! I’m a bit lost why it didn’t work.. I set it up on two independent instances and didn’t work on both. Now I started again from scratch and works on both! :+1: :upside_down_face:

2 Likes
25

Hi, about the audience, it seems impossible to target only TL0 users is it? Because everyone is TL0 ?

1 Like
26

I’m afraid you are correct @patrickemin The trust levels are additive and eventually it makes all logged-in users part of TL0 by default, even though if they are manually bumped up to TL4.

So, currently if you select TL0 as the audience, basically you are selecting all logged-in users. In itself it has a value IMHO, but this could be misleading.

I think I am going to update the logic for the audience field, and define a new rule for all Trust Levels. Thanks for giving me the idea : )

2 Likes
27

Hi @gormus I’ve followed your notes here for your response to nolo but for some reason I can’t get the notifications to run as a carousel. They just sit as stacked messages. I have two test messages and carousel is enabled for both and they sit in the same outlet. Nothing in the console for errors related to js. and I disabled all other components and nothing appears to work. Any ideas?

28

Hi @marksalter , sounds like you already checked out the usual suspects. Only other thing I can think of is trying to reproduce the issue using your setting exports. Pls DM your export file, sanitize the data if necessary. However I won’t be back for another 10 days : )

29

Hi I have this error when updating my banners message:

The JSON pointer property “/0/enabled_groups” must be an array of valid group IDs.

30

By any chance you have a reference to a deleted (or renamed) user group in the Audience field?

1 Like
31

Thanks that was that. I assumed groups were identified by their ID only and the name did not matter.

1 Like