Skip to content

Limited poll title character length to 250#115

Merged
BigSpaceships merged 2 commits into
ComputerScienceHouse:devfrom
FlyingCowMaster:dev
May 20, 2026
Merged

Limited poll title character length to 250#115
BigSpaceships merged 2 commits into
ComputerScienceHouse:devfrom
FlyingCowMaster:dev

Conversation

@FlyingCowMaster
Copy link
Copy Markdown

What

The title for polls are now limited to 250 characters.

Why

To prevent polls with the bee movie script as their title.

Test Plan

I tried to put the bee movie script as a poll title. It was capped to 250 characters.

Env Vars

environment variables were not changed

Documentation

Documentation was unchanged.

Checklist

  • [X ] Tested all changes locally

@BigSpaceships
Copy link
Copy Markdown
Contributor

I left a note on slack, but this is fantastic!
It would be great if it also validated this on the backend too so people can't spoof requests and post bee movie scripts (because I know cole will). You don't have to do weird frontend error validation because the frontend already limits it so if someone's messing with the api they can suffer

Also don't worry about sonarqube failing

@FlyingCowMaster
Copy link
Copy Markdown
Author

I have added backend validation, so now if the title is too long it returns a bad request.

Copy link
Copy Markdown
Contributor

@BigSpaceships BigSpaceships left a comment

Choose a reason for hiding this comment

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

goated :)

@BigSpaceships BigSpaceships linked an issue May 20, 2026 that may be closed by this pull request
@BigSpaceships BigSpaceships merged commit cd425fa into ComputerScienceHouse:dev May 20, 2026
1 of 2 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.

Add character limit for poll titles

2 participants