Skip to content

Adding Word Limit validation - #2

Open
glenblosser-nava wants to merge 6 commits into
mainfrom
gblosser/11025-add-word-limit-validator
Open

Adding Word Limit validation#2
glenblosser-nava wants to merge 6 commits into
mainfrom
gblosser/11025-add-word-limit-validator

Conversation

@glenblosser-nava

Copy link
Copy Markdown
Collaborator

Summary

Adds WordLimit support to validators.

Context for reviewers

Uses a similar structure to Length, but applies to words rather than character count. Uses a /\s+/ regex to identity words.

Comment thread backend/grants_shared/src/grants_shared/api/schemas/extension/field_validators.py Outdated
Comment thread backend/grants_shared/src/grants_shared/api/schemas/extension/field_validators.py Outdated

@chouinar chouinar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

To make this code available, we'll need to create & publish a new version.

For your PR, you'll just need to run make bump-patch-version which should update the pyproject & uv.lock file. After your PR is merged, we just need to run the github action that'll publish it to https://pypi.org/project/grants-shared/#description - and then we can update the dependency we import in the given service.

Comment thread backend/grants_shared/src/grants_shared/api/schemas/extension/field_validators.py Outdated
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.

2 participants