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.

Sign up for GitHub

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

Jump to bottom

2000 limit on replies #77

Closed
ldemailly opened this issue Aug 16, 2024 · 1 comment · Fixed by #78
Closed

2000 limit on replies #77

ldemailly opened this issue Aug 16, 2024 · 1 comment · Fixed by #78

Comments

@ldemailly
Copy link
Member

Ultimately we could attach the full result but for now let's just truncate
@ldemailly ldemailly mentioned this issue Aug 16, 2024
Merged
@ldemailly
Copy link
Member Author

ldemailly commented Aug 16, 2024 •
edited
Loading

@ccoVeille was getting

{
  "ts": 1723845161.081822,
  "level": "err",
  "r": 32,
  "file": "bot.go",
  "line": 286,
  "msg": "error",
  "err": "HTTP 400 Bad Request, {\"message\": \"Invalid Form Body\", \"code\": 50035, \"errors\": {\"content\": {\"_errors\":
 [{\"code\": \"BASE_TYPE_MAX_LENGTH\", \"message\": \"Must be 2000 or fewer in length.\"}]}}}"
}

so yeah 2000 is kinda "magic"

ldemailly added a commit that referenced this issue Aug 16, 2024
@ldemailly
Use runes - better fix for #77, review comments
64efcbb
ldemailly added a commit that referenced this issue Aug 16, 2024
@ldemailly
Use runes - better fix for #77, review comments (#79)
1256b8d 
* Use runes - better fix for #77, review comments

* add missing 'characters' word

* get the latest grol language version with * on slice and optional join/split sep arg

* review comment

* fix up english
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Truncate (for now) too long output grol-io/grol-discord-bot
1 participant
@ldemailly