-
-
Notifications
You must be signed in to change notification settings - Fork 132
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
feat(validate, http)!: add regex_patterns
and allow_list
#2189
feat(validate, http)!: add regex_patterns
and allow_list
#2189
Conversation
…t-rs#2161) Add support for the new `custom_message` field available on AutoMod `BLOCK_MESSAGE` actions. Discord API Docs: discord/discord-api-docs#5955 (cherry picked from commit 94ab90d)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Will approve this PR when the conflicts are resolved.
…ation # Conflicts: # twilight-http/src/request/guild/auto_moderation/create_auto_moderation_rule.rs # twilight-validate/src/request.rs
regex_patterns
and allow_list
regex_patterns
and allow_list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you for working on this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you for the pr! we should discuss whether we should move the requests to another pr or solve them here
twilight-http/src/request/guild/auto_moderation/create_auto_moderation_rule.rs
Show resolved
Hide resolved
twilight-http/src/request/guild/auto_moderation/create_auto_moderation_rule.rs
Show resolved
Hide resolved
twilight-http/src/request/guild/auto_moderation/create_auto_moderation_rule.rs
Show resolved
Hide resolved
twilight-http/src/request/guild/auto_moderation/create_auto_moderation_rule.rs
Show resolved
Hide resolved
twilight-http/src/request/guild/auto_moderation/create_auto_moderation_rule.rs
Show resolved
Hide resolved
In my opinion it is a bit too much for this pr. Also rewriting all this would probably take me a lot of time because I'm not that familiar with this crate. |
thats ok, we can merge this and create another pr for those changes |
This PR adds regex_patterns and allow_list when creating a auto moderation rule, as well as validation for most fields.
I'm pretty new to rust and this crate, so please provide lots of feedback, thanks :D