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

fix: modify_guild_role_positions #732

Merged
merged 1 commit into from
Dec 5, 2022
Merged

fix: modify_guild_role_positions #732

merged 1 commit into from
Dec 5, 2022

Conversation

Wolfhound905
Copy link
Collaborator

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

Discord expects an array here. This is a simple fix that makes the function work.

Changes

  • Wrap the payload in a list

Test Scenario(s)

await bot.http.modify_guild_role_positions(...)

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

@LordOfPolls LordOfPolls merged commit 12c27f0 into NAFTeam:dev Dec 5, 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
@Wolfhound905 @silasary @LordOfPolls