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

fix: Add delete_message_seconds to ban helpers #629

Merged
merged 1 commit into from
Sep 8, 2022
Merged

fix: Add delete_message_seconds to ban helpers #629

merged 1 commit into from
Sep 8, 2022

Conversation

zevaryx
Copy link
Contributor

@zevaryx zevaryx commented Sep 8, 2022

What type of pull request is this?

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

Description

Discord is deprecating delete_message_days on bans in favor of delete_message_seconds

Changes

  • Add delete_message_seconds field to ban helper functions in Guild and User
  • Add deprecation warning on delete_message_days value not being MISSING
    • Convert delete_message_days value to delete_message_seconds if used

Checklist

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

@LordOfPolls LordOfPolls merged commit 274b819 into dev Sep 8, 2022
@LordOfPolls LordOfPolls deleted the delete_message_seconds branch September 8, 2022 21:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@LordOfPolls LordOfPolls LordOfPolls approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zevaryx @LordOfPolls