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

quickSnip! #3084

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

quickSnip! #3084

wants to merge 5 commits into from

Conversation

Err0r430
Copy link

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

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.
Comment on lines 400 to 402
this.listener = addPreSendListener((context, msg) => {
console.log("Message content:", msg.content);
});
Copy link
Contributor

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?

Copy link
Author

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.
},
stop() {
removeChatBarButton("QuickSnip");
removePreSendListener(this.listener);
Copy link
Contributor

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

Copy link
Author

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.

Copy link
Contributor

@sadan4 sadan4 left a 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

@sadan4
Copy link
Contributor

sadan4 commented Jan 2, 2025

i feel like this would be better as an addition to message tags instead of its own plugin

@Err0r430
Copy link
Author

Err0r430 commented Jan 2, 2025

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 feel like this would be better as an addition to message tags instead of its own plugin

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.

@sadan4
Copy link
Contributor

sadan4 commented Jan 2, 2025

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

@Err0r430
Copy link
Author

Err0r430 commented Jan 2, 2025

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.

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.

3 participants