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

Send better error messages for 'ValidationError's #225

Merged
merged 2 commits into from
Jan 9, 2024

Conversation

wookie184
Copy link
Contributor

Provide better messages on the error pages for some validation errors.

Some of these shouldn't be reachable through the website so aren't that important, but ones like "Sum of file sizes exceed size limit when syntax highlighting applied" are good feedback to get as the reason for failure isn't otherwise clear. (PR inspired by a user on Python Discord wondering why they were getting a 400 error in this case)

Copy link
Owner

@supakeen supakeen left a comment

Choose a reason for hiding this comment

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

Yea, since these end up with end users it's a good idea to make them more clear.

@supakeen supakeen merged commit 85da347 into supakeen:master Jan 9, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants