Skip to content

feat: add fluentValidation validators for the min-max values#284

Merged
fvanderflier merged 12 commits intomainfrom
feature/add-min-max-validators
Jan 29, 2026
Merged

feat: add fluentValidation validators for the min-max values#284
fvanderflier merged 12 commits intomainfrom
feature/add-min-max-validators

Conversation

@mihailkitrozoskicodechem
Copy link
Collaborator

@mihailkitrozoskicodechem mihailkitrozoskicodechem commented Jan 23, 2026

What Changed?

Added FluentValidation rules to enforce valid min–max value constraints and prevent invalid configurations at the API level.

Added xUnit tests for the DomainModelEditor.

Why?

To prevent invalid data - minimum and maximum values from being submitted to the API.
Closes

Type of Change

  • 🐛 Bug fix (fixes an issue without breaking existing functionality)
  • ✨ New feature (adds new functionality without breaking existing features)
  • 💥 Breaking change (existing functionality will be affected)
  • 📝 Documentation update
  • 🔧 Maintenance (refactoring, dependencies, configuration)

How Did You Test This?

What I tested:

How to test it:

Review Checklist

  • My code follows the project's style and conventions
  • I've reviewed my own code for obvious issues
  • [] I've added comments where the code might be confusing
  • [] I've updated relevant documentation (if needed)
  • I've tested my changes and they work as expected

Screenshots / Additional Context

@fvanderflier fvanderflier changed the title feature/add fluentValidation validators for the min-max values feat: add fluentValidation validators for the min-max values Jan 23, 2026
@mihailkitrozoskicodechem
Copy link
Collaborator Author

TODO: Create tests for the fluentValidation Validators for the Min-Max values

@fvanderflier fvanderflier merged commit fa39a11 into main Jan 29, 2026
2 checks passed
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.

3 participants