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

fix: add in missing rate_limit_per_user #738

Merged
merged 1 commit into from
Dec 17, 2022
Merged

fix: add in missing rate_limit_per_user #738

merged 1 commit into from
Dec 17, 2022

Conversation

AstreaTSS
Copy link
Contributor

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change
  • CI change
  • Other: [Replace with a description]

Description

So someone forgot a line... and I found out the hard way.

Changes

  • Add in rate_limit_per_user for the super().edit for GuildText.

Test Scenario(s)

Just take any "regular" text channel in a guild and try editing rate_limit_per_user.

Checklist

  • I've formatted my code with Black
  • I've added docstrings to everything I've touched
  • I've ensured my code works on Python 3.10.x
  • I've ensured my code works on Python 3.11.x
  • I've tested my changes
  • I've added tests for my code - if applicable
  • I've updated the documentation - if applicable

silasary
silasary approved these changes Dec 17, 2022
View reviewed changes
Copy link
Collaborator

@silasary silasary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh no

@LordOfPolls LordOfPolls merged commit b137e84 into NAFTeam:dev Dec 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@silasary silasary silasary 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.
3 participants
@AstreaTSS @silasary @LordOfPolls