Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

fix: make fields for automod action optional #609

Merged

Conversation

AstreaTSS
Copy link
Contributor

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change

Description

This PR goes in to compare the Discord docs with the current AutoModerationAction to (try to) ensure every field that can be optional is marked as so.

Changes

  • Make all fields marked as optional on the Discord documentation optional in NAFF.
    • Channels were adjusted so that they report they are optional, too.
    • Since messages are optional, the message property has also been marked as optional.

Checklist

  • I've formatted my code with Black
  • I've ensured my code works on Python 3.10.x
  • I've tested my code

This is based on the Discord docs, and should hopefully be the last
time a bug like this appears.
Hopefully.
@silasary silasary added the Bug::Library A bug in the library label Aug 18, 2022
@silasary silasary merged commit 98b99f8 into NAFTeam:dev Aug 18, 2022
@AstreaTSS AstreaTSS deleted the automod-please-let-this-be-the-final-fix branch August 19, 2022 01:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug::Library A bug in the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants