Skip to content

London | November 25 | Donara Blanc | Sprint 1 | Fix bloom length limit#168

Open
donarbl wants to merge 1 commit intoCodeYourFuture:mainfrom
donarbl:sprint1-fix-bloom-length
Open

London | November 25 | Donara Blanc | Sprint 1 | Fix bloom length limit#168
donarbl wants to merge 1 commit intoCodeYourFuture:mainfrom
donarbl:sprint1-fix-bloom-length

Conversation

@donarbl
Copy link

@donarbl donarbl commented Feb 27, 2026

Self checklist

  • [x ] I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

Changed content TEXT to content VARCHAR(280) in schema.sql for a limit at database level
Kept length validation only in endpoints.py

@donarbl donarbl added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Legacy-Code The name of the module. labels Feb 27, 2026
Copy link

@OracPrime OracPrime left a comment

Choose a reason for hiding this comment

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

Nice work. Only one I've seen to update the schema as well.

@OracPrime OracPrime added Reviewed Volunteer to add when completing a review with trainee action still to take. Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed. Module-Legacy-Code The name of the module. Reviewed Volunteer to add when completing a review with trainee action still to take.

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants