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

added /quote command #860

Merged
merged 21 commits into from
Jul 5, 2024
Merged

added /quote command #860

merged 21 commits into from
Jul 5, 2024

Conversation

T1biriy
Copy link
Contributor

@T1biriy T1biriy commented Jun 3, 2024

/quote returns a random quote of a SAC member in a public channel - let SAC's wisdom be upon thee!

Added by Glaze so it shows up in the PR list properly:

To do:

  • Add catch for if message can't be found that displays message- and channelID so it can be removed from json.
  • Decide on (and implement way) to display message- and channelID on succesfull quote. Ideas:
    • Add button that sends an ephemeral message to the clicker containing this info.
    • Add it as a field/footer to the embed.
  • Test functionality before merging.
  • Add Shinx server ID to guildIDs.

Might add some more things here later.

created quote.js and added first batch of quotes
fixed command
…only

finalized first batch of quotes, added cooldown and made command SAC only
@Glazelf Glazelf self-assigned this Jun 3, 2024
@Glazelf Glazelf added enhancement New feature or request low priority will fix (later) labels Jun 3, 2024
Also some minor formatting
Probably? it boots.
Move quotes into seperate JSON file
Rework some logic
Start work on an embed
@Glazelf
Copy link
Owner

Glazelf commented Jun 17, 2024

Edited OP a bit to include a to-do list.

@Glazelf Glazelf marked this pull request as draft June 17, 2024 19:37
@Glazelf Glazelf marked this pull request as ready for review July 5, 2024 10:50
Copy link
Owner

@Glazelf Glazelf left a comment

Choose a reason for hiding this comment

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

Haven't tested this. Ideally in the future we make a testing Quotes object with message in the test server to test this on. For now, though, whatever. I'm sure it works perfectly!!!!!!!!

@Glazelf Glazelf merged commit 59ad93d into Glazelf:master Jul 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request low priority will fix (later)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants