Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Python Discord's plugins to the registry #3079

Merged
merged 2 commits into from
Aug 6, 2021

Conversation

Akarys42
Copy link

@Akarys42 Akarys42 commented Aug 2, 2021

This PR adds the following plugins:

  • Case insensitive snippets: Allow snippets to be ran even if with the wrong case. For example, ?Dm-RePort will be recognized as ?dm-report.
  • Close message: Add a ?closemessage command that will close the thread after 15 minutes with a default message.
  • MDLink: Generate a ready to paste link to the thread logs.
  • Reply cooldown: Forbid you from sending the same message twice in ten seconds.
  • Tagging: Add a ?tag command capable of adding a $message| header to the channel name.

The source code can be found in our repository. I have the authorisation from all the authors to create this PR.

- **Case insensitive snippets**: Allow snippets to be ran even if with the wrong case. For example, `?Dm-RePort` will be recognized as `?dm-report`.
- **Close message:** Add a `?closemessage` command that will close the thread after 15 minutes with a default message.
- **MDLink**: Generate a ready to paste link to the thread logs.
- **Reply cooldown**: Forbid you from sending the same message twice in ten seconds.
- **Tagging**: Add a `?tag` command capable of adding a `$message|` header to the channel name.
@RealCyGuy
Copy link
Member

?closemessage can be achieved with ?alias add closemessage close 15m Feel free to open a new thread if you need anything else.
?mdlink is just ?loglink with some formatting.
Everything else seems fine though.

@Akarys42
Copy link
Author

Akarys42 commented Aug 2, 2021 via email

@RealCyGuy
Copy link
Member

I think those two plugins are too niche but the other plugins look more useful. If someone wants to install those plugins, they could always install it the long way.

@lorenzo132
Copy link
Member

i dont see any reason why those plugins cannot be added @RealCyGuy

@fourjr
Copy link
Collaborator

fourjr commented Aug 6, 2021

Hey, thanks for the contribution.

We will be rejecting the following plugins:

@fourjr fourjr merged commit ae531c4 into modmail-dev:master Aug 6, 2021
@Akarys42
Copy link
Author

Akarys42 commented Aug 6, 2021 via email

@fourjr
Copy link
Collaborator

fourjr commented Aug 6, 2021

This should be live as soon as the registry cache expires. Might be live now, you can test it out on your instance with ?plugin registry mdlink

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants