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

fix a totally harmless feature #364

Merged
merged 1 commit into from
Apr 8, 2024
Merged

fix a totally harmless feature #364

merged 1 commit into from
Apr 8, 2024

Conversation

taranvohra
Copy link
Member

No description provided.

Copy link

what-the-diff bot commented Apr 8, 2024

PR Summary

  • Simplification of conditional checks in troll.ts
    The code that checks conditions for activating certain features has been simplified. It no longer requires the sender to be a staff member for the program to respond to messages starting with "skillissue".

  • Refactoring in troll.ts
    The process of extracting the guild, channel, and message IDs from the message link has been made more efficient and organized. This should improve the overall performance.

  • Added staff check before sending replies
    A new feature has been added that ensures only messages from staff members will get the "skill issue" reply. This keeps our responses more focused and decreases unnecessary interaction.

@taranvohra taranvohra merged commit eac2360 into main Apr 8, 2024
3 checks passed
@taranvohra taranvohra deleted the fix-troll branch April 8, 2024 17:48
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.

2 participants