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.

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

fix: limit thread name #1733

Merged
merged 1 commit into from
Jan 17, 2025
Merged

Conversation

desaxce
Copy link
Collaborator

@desaxce desaxce commented Jan 17, 2025

Fix for bug reported in this issue.
No need to limit var length on Prisma schema itself.

To test, start a thread with a text exceeding 255 characters, make sure you don't get the error below and you can still search through your threads efficiently.

Error that should not appear anymore:

2025-01-17 13:11:19 - Database error: index row requires 204672 bytes, maximum size is 8191
2025-01-17 13:11:19 - Error updating thread: index row requires 204672 bytes, maximum size is 8191

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. bug Something isn't working data layer Pertains to data layers. labels Jan 17, 2025
@willydouhard willydouhard merged commit fb1e646 into main Jan 17, 2025
9 checks passed
@willydouhard willydouhard deleted the hugues/limit-thread-name-to-255-chars branch January 17, 2025 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working data layer Pertains to data layers. size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants