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

add thanks command, update env, msg package and readme #1

Closed
wants to merge 1 commit into from

Conversation

devldm
Copy link
Owner

@devldm devldm commented Feb 19, 2024

📝 Description

Adding a ThankCommand and a thanks channel. This is in the hopes of making it easier to acknowledge people that have helped you as well as providing a place for "appreciation messages" to live within the server.

This should both help users see who is helping others whilst also promoting helping others as its now more visibly rewarded / celebrated.

✔️ PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Chore (non-breaking change which does not affect codebase; Build related changes, version modification, etc.)

🎯 Current behaviour

Currently there is no way to explicitly thank another user outside of leaving a thanks message or emoji.

🚀 New behaviour

1. Use the /thanks command as shown below

image

2. Upon sending the bot responds with

image

3. The bot then sends a message to a newly created thanks channel as below

image

Things to consider

  • This implementation would require a new channel or re-purposing of an existing channel as a place to send these messages to.
  • A THANKS_CHANNEL id will need to be added to local .envs in order to test and push out this feature as reflected in the updated readme.md
  • While in these changes LogThanks has been added, this may be seen to send logs unnecessarily and could be removed without issue
  • This command could likely be used in the future to send POST requests to a potential thank-track database as mentioned in the project-showcase channel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant