Skip to content
This repository has been archived by the owner on Oct 6, 2024. It is now read-only.

Emotes - add create/list/delete slash commands #22

Merged
merged 5 commits into from
Mar 1, 2024
Merged

Conversation

lenisko
Copy link
Collaborator

@lenisko lenisko commented Feb 29, 2024

Add ability to create, list and delete (only created by bot) emotes using slash commands (admin only)

@TMHBOFH
Copy link

TMHBOFH commented Feb 29, 2024

please also update the Dockerfile

COPY default.toml /porygon/
+ COPY /emojis /porygon/emojis
COPY /templates /porygon/templates

thanks

@lenisko
Copy link
Collaborator Author

lenisko commented Mar 1, 2024

@TMHBOFH I'm not an Docker expert, but you don't want to put those files in image but bind them. Check compose example.

@TMHBOFH
Copy link

TMHBOFH commented Mar 1, 2024

@lenisko
I know what you mean, but would it run if there wasn't even an empty emojis folder?

As I understand it, at least the empty emojis folder schuld be in the image or?

@lenisko
Copy link
Collaborator Author

lenisko commented Mar 1, 2024

@TMHBOFH it will start just fine, those are required once you want to run create-emotes command.

@lenisko lenisko changed the title feat: add create/list slash commands feat: add create/list/delete slash commands Mar 1, 2024
@lenisko lenisko changed the title feat: add create/list/delete slash commands Emotes - add create/list/delete slash commands Mar 1, 2024
Copy link

@na-ji na-ji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, the commands work well. It does the job. Now the next step would be to also use the emojis that were uploaded. :)

@lenisko
Copy link
Collaborator Author

lenisko commented Mar 1, 2024

Thanks :)

Yeah, fairly easy to create a mapping of emote name <> config and return it in message. Good for someone else!

@lenisko lenisko merged commit 09f3aca into main Mar 1, 2024
1 check passed
@lenisko lenisko deleted the create_list_cmd branch March 1, 2024 21:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants