Skip to content

London | 25-SDC-Nov | Zohreh Kazemianpour | Sprint 1 | Debugging - Extra long blooms#141

Open
zohrehKazemianpour wants to merge 2 commits intoCodeYourFuture:mainfrom
zohrehKazemianpour:extra-long-bloom
Open

London | 25-SDC-Nov | Zohreh Kazemianpour | Sprint 1 | Debugging - Extra long blooms#141
zohrehKazemianpour wants to merge 2 commits intoCodeYourFuture:mainfrom
zohrehKazemianpour:extra-long-bloom

Conversation

@zohrehKazemianpour
Copy link

Learners, PR Template

Self checklist

  • 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

I added server-side validation to enforce the 280-character limit on blooms to prevent posts longer than the limit from being saved to the database.

@zohrehKazemianpour zohrehKazemianpour added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 16, 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.

If I were being picky, I'd say "280 characters or fewer" but this is a programming test not an English grammar test, so I won't :)

@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 19, 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. 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

Comments