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

Implemented a REGEX check for <@ID> #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JayGaba
Copy link

@JayGaba JayGaba commented Oct 13, 2024

Closes #10

This pull request includes updates to the trigger statements in the issues.ts and ping.ts files to improve the accuracy of text matching.

Changes in trigger statements:

  • triggers/issues.ts: Updated the statement in the IssuesTrigger filter to use a regular expression for more precise matching of the text pattern.
  • triggers/ping.ts: Updated the statement in the pingTrigger filter to use a regular expression for more precise matching of the text pattern.Fixed the issue using regex check since bot id is not a guaranteed field in the event data

Fixed the issue using regex check since bot id is not a guaranteed field in the event data
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.

Make ping and issues triggers check for more specific messages
1 participant