Alternar Whisper 👁

Componente de tema
1

This theme component adds a post menu button for staff users to make a regular post a whisper or vice versa.

:wrench: How to install

:computer: Get the code

Choose the eye-slash icon in the post controls to make the post a whisper
Choose the eye-slash icon in the post controls to make the post a whisper883×293 17.6 KB

Choose the eye icon in the post controls to make the post a regular post
Choose the eye icon in the post controls to make the post a regular post883×293 18 KB

This is useful especially for forums where a lot of conversations happen in whisper and a staff user ends up posting something and later wants to hide or show the post to non-staff users.

30 curtidas
2

Nice! I remember this debate a few years ago. I have been burned by this so many times. The official response is:

I am often scrambling to delete whispers before they get emailed out and too many people notice my mistake!

9 curtidas
3

Brilliant component!

is this possible to just make it so it appears on staff posts?

4 curtidas
4

Thank You @fzngagan

Well done!

1 curtida
5

I’ll take a look at this request soon.

p.s.
This is addressed now.

2 curtidas
6

Yes I think the same, it’s very easy to get wrong with whispers and this is one of the cases where simplicity has too be sacrified for peace of mind.

5 curtidas
7

My question then is, does discourse behave differently in terms of email if the post is deleted vs when its post_type is changed?

1 curtida
8

We have this component installed, but the toggle no longer shows up.

Has it stopped working, or is it possible that we made some edit elsewhere (to our themes?) that inadvertently blocked it?

9

It’s definitely still working on meta.

What does your setting here say:

image
image2304×530 50.9 KB

1 curtida
10

Whispers are allowed

Selection_1065
Selection_1065763×155 15.6 KB

17

Ah I misread the original issue, it’s not about the general whisper functionality, gotcha! @fzngagan is already on the case it seems :wink:

1 curtida
18

Apparently this code is broken. siteSettings.enable_whispers has been modified to siteSettings.whisper_allowed_groups.

You can try my fork here:

3 curtidas
19

Thanks @Lhc_fl! Unfortunately, I don’t think I can try your fork since we’re on a hosted instance. But I appreciate your quick investigation! Maybe that will speed up the official fix. :awthanks:

1 curtida
20

You can have any theme component you like. :+1: :slight_smile: It’s only plugins that need to be official (or vetted)

However, I think @fzngagan will want to fix this one up, so if you can wait it may be better in this case.

2 curtidas
21

It’s actually a pavilion repo

@Lhc_fl PR and I will review.

2 curtidas
22

Do you want to add the pavilion tag?

1 curtida
23
5 curtidas
24

Yeah let’s do that and I’ll discuss if @fzngagan wants to pass the Topic over.

1 curtida
26

@merefield Is there any movement on this? I think it may still be in limbo? :slight_smile:

3 curtidas
27

It’s waiting on a (presumably trivial) org decision. I chased today.

2 curtidas