add length validation to description field

[bkmorgan3]

Fixes #1363

What changes did you make and why did you make them ?

• added regex value for all characters, max of 250

[github-actions]

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b bkmorgan3-1363/Description-validation development
git pull https://github.com/bkmorgan3/VRMS.git 1363/Description-validation

[MattPereira]

Since the ProjectForm component only controls creating new projects the current state of this PR does not stop existing projects from being updated to have > 250 character descriptions

[bkmorgan3]

@MattPereira I see. I assumed it was lumped in with the other new project validation / react hook form work. I'll refactor

@spiteless Whats the best path forward here? Its kind of an anti pattern to have to forms written two different ways. Do we edit the ticket, use one form component, or edit the scope ?

[trillium]

We're going toward getting both the Create and Edit pages being derived from one component with validations passed to the root component (#1471 does this now). Happy to get this validation working and then extend it across the project properly.

[trillium]

Waiting to merge #1471 first, then this PR will need to be rebased

[bkmorgan3]

@spiteless updated.

[trillium]

Hey @bkmorgan3 , can you redo these changes on the current version of development. The file location has moved to client/src/components/data.js and is creating merge conflicts that we can sidestep

[trillium]

Can you redo these changes on the current version of dev in the new client/src/components/data.js file and force push?

[bkmorgan3]

@spiteless This update to dev was done after my pr was submitted. done.

[trillium]

Silly me, I thought there'd be conflicts here but there are not