-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
quickSnip! #3084
base: main
Are you sure you want to change the base?
quickSnip! #3084
Conversation
A quick preset snippet tool for sending those repetitive messages. Great for mods and admins of large servers who have to answer the same questions over and over again.
src/plugins/quickSnip/index.tsx
Outdated
this.listener = addPreSendListener((context, msg) => { | ||
console.log("Message content:", msg.content); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems you left some unnecessary logging in?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching it, my bad for not removing it.
From: MultiAccountStore in localstorage. To: UserStore. Removed unused console.log from testing. Handled bug if we arent in DMS.
src/plugins/quickSnip/index.tsx
Outdated
}, | ||
stop() { | ||
removeChatBarButton("QuickSnip"); | ||
removePreSendListener(this.listener); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You no longer have this listener
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated that, I'm confused why my eslint has decided to not work in the slightest.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs a readme.md
i feel like this would be better as an addition to message tags instead of its own plugin |
Added the README as requested. I'm unsure of the format of these (I took a look at a few but wasn't consistent) so I just wrote one that highlighted its features.
I considered this, but I felt as though messageTags is geared towards a more keyboard-based usage experience, whereas QuickSnip is a simple button interface aimed at server admins/mods who repeatedly send the same basic messages to users. |
i see no reason not to combine the two, sometimes I don't want to type a command / would rather type a command. plus it would be confusing having snippets in two separate places |
So shall I merge this plugin with messageTags? I still feel as though this could be a standalone plugin but am happy to merge if you think so. |
A quick preset message snippet tool for sending those repetitive messages. Great for mods and admins of large servers who have to answer the same questions over and over again with the exact same answer. It has a base level of support for when in dms replacing ${myName} with your username and ${userName} with their name (I'll definitely be adding more of these inline replacement things in the future).