revanced-helper icon indicating copy to clipboard operation
revanced-helper copied to clipboard

feat: support mute

Open Ushie opened this issue 2 years ago • 24 comments

Superseding #15 as the discussion there has gotten too long and hard to follow

Support Mute is now implemented in ReVanced Helper, under the name of "exile", but it's exclusively a slash command, given the context of the motivation for this command to exist in the first place, it's more ideal for it to be available in the context menu, that'd be easier for rapid fire and it also skips having to manually specify a message to delete

I'm also not sure whether this was implemented or not, but the message that was specified should also be moved to the support channel with this format:

"@Ushie said: why is revanced stopping at 1:09"

This ensures that the message is kept and the user doesn't have to write their message twice, and also helps in telling the user where they should go as they now have a ping in the support channel

The commands would follow this structure: "Support mute" in context menu, which does the following:

  1. give the user the support muted role
  2. moves the user's message to #support in the structure shown above
  3. delete the initial message in chat

?smute as a command that depends on replying to messages:

  1. the same 3 steps above
  2. delete the message containing the ?smute command

Why reply? it's faster and specifies the message that should be moved and deleted

Ushie avatar Aug 10 '23 12:08 Ushie

If rapid fire is the goal, then slash commands would be quicker as you can copy and paste, then ping the user. The distance for your finger to travel is shorter.

But we cant select which msg to redirect.

SodaWithoutSparkles avatar Aug 11 '23 21:08 SodaWithoutSparkles

s more ideal for it to be available in the context menu, that'd be easier for rapid fire and it also skips having to manually specify a message to delete

On mobile it may be faster to use slash commands, both forms may exist as a way to trigger the command

oSumAtrIX avatar Aug 11 '23 21:08 oSumAtrIX

Why reply?

It would sure be nice to implement a regular text command with reply parsing. This was mentioned in the last issue but Reis had this to say

taku-nm avatar Aug 13 '23 13:08 taku-nm

Why reply?

It would sure be nice to implement a regular text command with reply parsing. This was mentioned in the last issue but Reis had this to say

That is because it was being referred to as a prefix command, with parameters and everything, in this case, that's not necessary as replying is necessary, the bot can get the necessary details from the message that was replied to

Ushie avatar Aug 13 '23 14:08 Ushie

Proposal 3 Options to execute this command: (exile or smute doesn't matter)

(replied message) ?smute [no param] /smute [user] [msg count] context menu: support-mute [no param]

This way each supporter can choose whatever method is easiest to use depending on what device and platform they're currently on.

taku-nm avatar Aug 13 '23 14:08 taku-nm

I propose using stopit instead of smute to make it more interesting.

stop it, get some help

SodaWithoutSparkles avatar Aug 13 '23 14:08 SodaWithoutSparkles

When does the bot remove the role again?

oSumAtrIX avatar Aug 13 '23 15:08 oSumAtrIX

I'd suggest 5 or 10 minutes

taku-nm avatar Aug 13 '23 15:08 taku-nm

Should be configurable via the bot configs.

oSumAtrIX avatar Aug 13 '23 15:08 oSumAtrIX

Just to keep track of the current implementation and what's missing from it:

  • [x] Bot should delete the original message it copied to support (cut and paste principle)
  • [ ] Add attachments support to the redirected message
  • [ ] Bot should not be able to exile bots
  • [x] The logging of exile and unexile actions should happen in the logs thread
  • [ ] If a user is already exiled, don't allow exiling again
  • [ ] Bot should provide a reaction to click to unmute (effective acknowledgement)
  • [ ] No verbose output in the channel the command has been executed at, only report to log channel

taku-nm avatar Aug 13 '23 18:08 taku-nm

@taku-nm I've added checkboxes

oSumAtrIX avatar Aug 13 '23 18:08 oSumAtrIX

@taku-nm Why should bots not reply with help? Or what are you referring to?

oSumAtrIX avatar Aug 13 '23 18:08 oSumAtrIX

@taku-nm Why should bots not reply with help? Or what are you referring to?

It should not reply in cafe because why should it? User asks a question, gets exiled and can't read the bot reply anyways. One might make the bot interpret the reply it's cuting and pasting and automatically reply to it in support chat. @ User Your question: Proposed answer:

taku-nm avatar Aug 13 '23 18:08 taku-nm

I think he is refering to the "you do not have permission to do this" thing.

I propose, instead of sending a message, just react a 🙄 or ❌.

SodaWithoutSparkles avatar Aug 13 '23 18:08 SodaWithoutSparkles

The bot does log in logs and the bot does not reply in any channel unless you run a slash command. Also, I don't think the bot replying "You don't have the permission to do that" is much of a problem to add unnecessary code. It does not cover uo your whole screen.

reisxd avatar Aug 13 '23 18:08 reisxd

It should not reply in cafe because why should it?

To retain history/context. Messages randomly disappearing will be confusing for anyone participating the chat, instead it should show a message "Moved to X" or similar.

oSumAtrIX avatar Aug 13 '23 18:08 oSumAtrIX

To retain history/context.

What? I think we're talking about different things here. I'm saying that the rust bot should no longer reply to questions in cafe as they should not be answered / even asked there in the first place.

I guess technically this isn't part of this specific issue, but might be something you can quickly implement if the config for Rust bot allows for that

taku-nm avatar Aug 13 '23 19:08 taku-nm

What? I think we're talking about different things here. I'm saying that the rust bot should no longer reply to questions in cafe as they should not be answered / even asked there in the first place.

The rust bot is not part of this issue and is offtopic. If you expect any changes for the rust bot, please do so in the correct repository.

oSumAtrIX avatar Aug 13 '23 19:08 oSumAtrIX

I've just checked, this also applies to the helper bot tho. It still replies with with troubleshooting help outside the support channel.

taku-nm avatar Aug 13 '23 19:08 taku-nm

Are you refering to autoresponses?

oSumAtrIX avatar Aug 13 '23 19:08 oSumAtrIX

Are you refering to autoresponses?

Yes

taku-nm avatar Aug 13 '23 20:08 taku-nm

Specific auto-responses should still be sent in channels like #cafe. However, we are talking about a server-specific configuration, which is out of the scope of this issue.

oSumAtrIX avatar Aug 13 '23 20:08 oSumAtrIX

Any updates for this? Can someone summarize the whole discussion to a list of things I need to implement?

PalmDevs avatar Nov 25 '23 16:11 PalmDevs

https://github.com/ReVanced/revanced-helper/issues/19#issuecomment-1676429790

Ushie avatar Nov 25 '23 16:11 Ushie

Closed as it is no longer added. If needed again, can be added via the "role presets" feature.

PalmDevs avatar Aug 16 '24 05:08 PalmDevs