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

fix: body-length can't be exceeded anymore #744

Merged
merged 2 commits into from
Jan 5, 2023

Conversation

AlbertUnruh
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

Fix a bug where a body-length near 2000 can cause errors because formatting increases its length

Changes

Move formatting of body up so it's length is checked

Test Scenario(s)

You should see that it works just by looking at it /shrug

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 40254a0 into NAFTeam:dev Jan 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants