Skip to content

London | 26-SDC-Mar | Craig D'Silva | Sprint 1 | Extra long blooms#202

Open
craig-dsilva wants to merge 2 commits into
CodeYourFuture:mainfrom
craig-dsilva:bug/extra-long-blooms
Open

London | 26-SDC-Mar | Craig D'Silva | Sprint 1 | Extra long blooms#202
craig-dsilva wants to merge 2 commits into
CodeYourFuture:mainfrom
craig-dsilva:bug/extra-long-blooms

Conversation

@craig-dsilva

Copy link
Copy Markdown

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

Changelist

This adds bloom length validation on the backend.

How to test

  1. Login to Purple Forest.
  2. Open DevTools by right-clicking on the new bloom textarea element and selecting "Inspect".
  3. Change "maxlength" of the textarea to be greater than 280.
  4. Submit the bloom by clicking the "Bloom" button.
  5. A modal will be displayed with the error status 422 on the backend.

@craig-dsilva craig-dsilva added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 🏕 Priority Mandatory labels Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 🏕 Priority Mandatory 📅 Sprint 1 Assigned during Sprint 1 of this module

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant