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: Add POLL_RESULT to MessageType enum #1735

Merged
merged 6 commits into from
Sep 28, 2024

Conversation

LawMixer
Copy link
Contributor

@LawMixer LawMixer commented Sep 3, 2024

Pull Request Type

  • Feature addition
  • Bugfix
  • Documentation update
  • Code refactor
  • Tests improvement
  • CI/CD pipeline enhancement
  • Other: [Replace with a description]

Description

This is a fix to the warning invalid/unexpected value 46 - the commit message is wrong input (accident);

Class `MessageType` received an invalid and unexpected value `46`, a new enum item will be created to represent this value. Please update interactions.py or report this issue on GitHub - https://github.com/interactions-py/interactions.py/issues

Changes

  • Added 46 POLL_RESULT to MessageType enum;

Related Issues

#1731

Test Scenarios

When MessageCreate is called on a poll_result it will give a warning that the message type doesn't exist;

Python Compatibility

  • I've ensured my code works on Python 3.10.x
  • I've ensured my code works on Python 3.11.x

Checklist

  • [x ] I've run the pre-commit code linter over all edited files
  • I've tested my changes on supported Python versions
  • I've added tests for my code, if applicable
  • I've updated / added documentation, where applicable

@silasary silasary enabled auto-merge (squash) September 21, 2024 05:08
@silasary silasary changed the base branch from stable to unstable September 28, 2024 05:37
@silasary silasary merged commit b030d65 into interactions-py:unstable Sep 28, 2024
2 checks passed
@silasary
Copy link
Member

Please remember to base your PRs on unstable in the future :)

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