~nova REPORTED IMPLEMENTED 1 year, 5 months ago

The preview-allowed preference now exists. This can be a guild or channel-scoped preference.

~nova assigned ~nova 1 year, 5 months ago

~nova assigned ~lockandkeys 1 year, 5 months ago

~nova 1 year, 5 months ago

Assigning keys for documentation of the new preference

~erin 1 year, 4 months ago*

This wasn't implemented as I'd thought it'd be 😖

Enabling preview and disabling preview-allowed disables the preview functionality for message links that point to my server, when posted to another server. It also disables the expansion of message links entirely on the server the setting is applied on. Even if the message originates from that server, or another.

What I had imagined, and thought was clear from prior discussion of this feature was that the desired restriction would only be when a message link was posted to another server. That is, message previewing, and everything else that preview enables would remain active on the server that desires this privacy option, but when the target message refers to the server in question, and the link wasn't posted on that server, an expansion wouldn't take place.

Would you please re-open this ticket?

~luma_inhibitor 1 year, 4 months ago*

I submitted a patch via email but ignore that one because it introduced a typo!!!

woof

diff --git a/messagefuncs.py b/messagefuncs.py
index 63fe0e8a..e6d7cc74 100644
--- a/messagefuncs.py
+++ b/messagefuncs.py
@@ -478,7 +478,7 @@ async def preview_messagelink_function(message, client, args):
                 ).read_message_history
             ):
                 return
-            if not config.get(
+            if message.guild != guild and not config.get(
                 key="preview-allowed", default=True, guild=guild, channel=channel
             ):
                 return