ReVanced / revanced-helper

🤖 NLP backed bots assisting ReVanced
https://revanced.app
GNU General Public License v3.0
20 stars 8 forks source link

feat: support mute #19

Closed Ushie closed 2 months ago

Ushie commented 1 year ago

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

SodaWithoutSparkles commented 1 year ago

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.

oSumAtrIX commented 1 year ago

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

taku-nm commented 1 year ago

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

Ushie commented 1 year ago

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

taku-nm commented 1 year ago

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.

SodaWithoutSparkles commented 1 year ago

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

stop it, get some help

oSumAtrIX commented 1 year ago

When does the bot remove the role again?

taku-nm commented 1 year ago

I'd suggest 5 or 10 minutes

oSumAtrIX commented 1 year ago

Should be configurable via the bot configs.

taku-nm commented 1 year ago

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

oSumAtrIX commented 1 year ago

@taku-nm I've added checkboxes

oSumAtrIX commented 1 year ago

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

taku-nm commented 1 year ago

@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:

SodaWithoutSparkles commented 1 year ago

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 ❌.

reisxd commented 1 year ago

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.

oSumAtrIX commented 1 year ago

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.

taku-nm commented 1 year ago

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

oSumAtrIX commented 1 year ago

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.

taku-nm commented 1 year ago

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

oSumAtrIX commented 1 year ago

Are you refering to autoresponses?

taku-nm commented 1 year ago

Are you refering to autoresponses?

Yes

oSumAtrIX commented 1 year ago

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.

PalmDevs commented 11 months ago

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

Ushie commented 11 months ago

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

PalmDevs commented 2 months ago

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