Skip to content

fix a totally harmless feature#364

Merged
taranvohra merged 1 commit intomainfrom
fix-troll
Apr 8, 2024
Merged

fix a totally harmless feature#364
taranvohra merged 1 commit intomainfrom
fix-troll

Conversation

@taranvohra
Copy link
Member

No description provided.

@what-the-diff
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
@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

Comments