Skip to content

CV2-6653: Articles :: Add max length limitation to match UI validation#2438

Open
melsawy wants to merge 3 commits into
developfrom
CV2-6653-articles-add-max-length-limitation-to-match-ui-validation
Open

CV2-6653: Articles :: Add max length limitation to match UI validation#2438
melsawy wants to merge 3 commits into
developfrom
CV2-6653-articles-add-max-length-limitation-to-match-ui-validation

Conversation

@melsawy

@melsawy melsawy commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Description

Add a validation to FactCheck and Explainer to verify the length for the article fields to match UI validation

  • Add validations per field (title, url and description)
  • Truncate fields for automation import (i.e user is a BotUser)

References: CV2-6653

How to test?

Added unit tests

Checklist

  • I have performed a self-review of my code and ensured that it is safe and runnable, that code coverage has not decreased, and that there are no new Code Climate issues. I have also followed Meedan's internal coding guidelines.

@melsawy melsawy changed the title CV2-6653: validate article fields length CV2-6653: Articles :: Add max length limitation to match UI validation Jun 18, 2026
@melsawy melsawy marked this pull request as ready for review June 18, 2026 15:29
@melsawy melsawy requested a review from caiosba June 18, 2026 15:29

@caiosba caiosba left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed in pair in our 1:1 ✔️

@melsawy melsawy requested a review from caiosba June 23, 2026 17:08
@melsawy

melsawy commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

@caiosba, I updated the PR to validate each field independently. I also added automatic truncation for these fields when the import is performed by a BotUser (automation import), rather than implementing truncation logic across all Zapier integrations.

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