Skip to content

fix: messages validation#4704

Merged
mariojsnunes merged 2 commits intomasterfrom
fix/messages-error-validation
Apr 4, 2026
Merged

fix: messages validation#4704
mariojsnunes merged 2 commits intomasterfrom
fix/messages-error-validation

Conversation

@mariojsnunes
Copy link
Copy Markdown
Contributor

@mariojsnunes mariojsnunes commented Apr 4, 2026

PR Checklist

  • - Unit and/or e2e tests for the changes that have been added (for bug fixes / features)

What kind of change does this PR introduce?

  • 🐛 Bugfix — fixes incorrect behavior without changing functionality

What is the new behavior?

  1. The contact messages form wasn't validating the message field.
  2. API also didn't check for min and max length.
  3. UI - Changed "Contact" button to "Send message" - Contact is already the form title.

Does this PR introduce a DB Schema Change or Migration?

  • No

@cypress
Copy link
Copy Markdown

cypress bot commented Apr 4, 2026

onearmy-community-platform    Run #9203

Run Properties:  status check passed Passed #9203  •  git commit af6b1555dd: chore: add ci error logging
Project onearmy-community-platform
Branch Review fix/messages-error-validation
Run status status check passed Passed #9203
Run duration 07m 59s
Commit git commit af6b1555dd: chore: add ci error logging
Committer Mário Nunes
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 95
View all changes introduced in this branch ↗︎

@mariojsnunes mariojsnunes merged commit c2063b1 into master Apr 4, 2026
12 checks passed
@mariojsnunes mariojsnunes deleted the fix/messages-error-validation branch April 4, 2026 16:22
@onearmy-bot
Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 2.117.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants