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

Add epehemerals #203

Closed
wants to merge 7 commits into from
Closed

Conversation

DRSchlaubi
Copy link
Contributor

This uses ephemeral for this command: (discussion is welcome)

Clarification for tags: info, delete, raw

Credits

CommandKiller: @Amejonah1200

- Also query alias for auto-help tags
- Increase tag column size since the message limit doesn't account for Markdown
- Make autohelp find casting tag properly
- Fix tag finding
Co-authored-by: amejonah1200 <amejonah@gmail.com>
@Amejonah1200
Copy link

I disagree to make rank commands ephemeral. How do I flex with my levels then? :(

@NyCodeGHG
Copy link
Contributor

@Amejonah1200 just take a screenshot of the output and send that into the channel and flex with that.

@Amejonah1200
Copy link

Amejonah1200 commented Jun 1, 2021

Well, I can just edit it with Ctrl+Shift+I

@DRSchlaubi
Copy link
Contributor Author

...

@dlsf
Copy link
Member

dlsf commented Jun 20, 2021

Rank, Tag and Blacklist command should not be ephemeral

@DRSchlaubi
Copy link
Contributor Author

Rank, Tag and Blacklist command should not be ephemeral

Why would the most annoying command which's entire purpose it is to show user-specific information be public

@dlsf
Copy link
Member

dlsf commented Jun 20, 2021

Why would the most annoying command which's entire purpose it is to show user-specific information be public

For moderation purposes. Additionally, blacklist and some tag commands are relevant for all users

@dlsf dlsf requested a review from a team June 20, 2021 13:41
@dlsf dlsf added the enhancement New feature or request label Jun 20, 2021
@DRSchlaubi
Copy link
Contributor Author

Why would the most annoying command which's entire purpose it is to show user-specific information be public

For moderation purposes. Additionally, blacklist and some tag commands are relevant for all users

I don't see how levels would be required for mod purposes, especially if you consider that a user can delete any command response using command responses just sounds like a lazy excuse not to make a proper log

@dlsf
Copy link
Member

dlsf commented Jun 20, 2021

Why would the most annoying command which's entire purpose it is to show user-specific information be public

For moderation purposes. Additionally, blacklist and some tag commands are relevant for all users

I don't see how levels would be required for mod purposes, especially if you consider that a user can delete any command response using command responses just sounds like a lazy excuse not to make a proper log

I'll work on a log but until then we can't use ephemerals.
Level messages are sometimes needed, for instance when restoring the data of a user. I agree, it's not that important but IMO it should be a normal message too (for now?).

@DRSchlaubi
Copy link
Contributor Author

the entire reason why epehmerals was being worked on is to make ranks ephemeral. There should be backups of the database using the output of /rank of a backup is just unprofessional

@DRSchlaubi
Copy link
Contributor Author

Will be superseeded soon

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants